Burton 2022 added

Changes pushed. Except for re-indexing behavior_2 and 3 which aren't set up
to be compatible with a stimulus index. Can be done with some
restructuring, but would be good first to know if there is any data in
those files that you don't want to actually keep.
On Wed, Jun 8, 2022 at 3:55 PM Richard C Gerkin @.***>
wrote:

@.**** requested changes on this pull request.

Also change CID to a stimulus ID everywhere (see stimuli.csv for
details).
Change behavior1 to use stimulus, subject (now experiment, but with
same info, i.e. first column of subjects.csv -- a subject is a hemisphere),
roi, dF.

In burton_2022/manifest.toml
https://github.com/pyrfume/pyrfume-data/pull/107#discussion_r892799686:

+extra = ""

+[raw]
+"raw/odormappingdata_fullinfo_06_2021.mat" = "All raw data except for odorant concentrations. See main.py for full details."
+"raw/odormappingdata_simple_05_2022.mat" = "Simplified version of 'fullinfo' .mat file containig deltaF response matrices and odorant concentrations. See main.py for full details."
+"raw/ID_glom_04_2022.mat" = "Data for functionally-identifiable glomeruli and diagnostic odorants. See main.py for full details."
+"raw/XXXX_ROIPos.csv" = "X and Y coordinates for the centroids of each ROI (glom) in each experiment XXXX; exported from odormappingdata_fullinfo_06_2021.mat file because Matlab 'table' class is not Python readable."
+"LICENSE" = ""
+
+[processed]
+"main.py" = "Processing workflow."
+"molecules.csv" = "Information about odorant molecules used. -1 CID assigned to 'empty' which is the control."
+"imaging.csv" = "Paths to response maps saved in .csv form for every odorant/mouse combination tested."
+"behavior_1.csv" = "DeltaF response values of every ROI (glomerulus) for every odorant/mouse tested."
+"behavior_2.csv" = "Properties of diagnostic odorants with functionally-identified glomeruli. See main.py for full details."
+"behavior_3.csv" = "Properties of diagnostic odorants that consistently elicit sparse activation but did not meet the cirteria for functional identification. See main.py for full details."
Typo in criteria.
In burton_2022/stimuli.csv
https://github.com/pyrfume/pyrfume-data/pull/107#discussion_r892802707:

@@ -0,0 +1,741 @@
+CID,Mouse ID,Conc. (mols/L)

Reorganize to have columns:
"Stimulus", "CID", and "Conc. (mols/L)"
Every unique combination of CID and concentration should have a different
stimulus ID (could be a simple concatenation with an underscore, up to a
certain number of digits).
Add a row for blank.
Drop Mouse ID.
In burton_2022/molecules.csv
https://github.com/pyrfume/pyrfume-data/pull/107#discussion_r892802919:

@@ -0,0 +1,188 @@
+CID,MolecularWeight,IsomericSMILES,IUPACName,name
+-1,,,,empty

Drop the -1.
—
Reply to this email directly, view it on GitHub
https://github.com/pyrfume/pyrfume-data/pull/107#pullrequestreview-1000353101,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMR2YW7R2ODBRMILVF2MYMLVOD3BZANCNFSM5YHHCY6A
.
You are receiving this because you authored the thread.Message ID:
@.***>


Disappearing thumbnails

When a project is closed, then reopened, some or all of the thumbnail images disappear. View > Reload / Force Reload doesn't resolve the issue. Screenshots attached.

Thanks for the report! I have not experienced this before. I will have a look at it.
If you re-open the app, do the thumbnails load properly again?

Nope. Thanks for looking into it! Let me know if you need any further info.


Prevent future run again after switching tab

So, i have multiple tab. For an example, i have two tabs, tab 1 and tab 2. At tab 1 there is a future that fetch my data, then i go to tab 2 and back again to tab 1 but the future is running again. Well, i;m trying to use
 bool get wantKeepAlive => true;
here is my script
Future<List<HotProducts>> _loadHotProducts;
  @override
  void initState() {
    _loadHotProducts = fetchHotProducts(http.Client());
    super.initState();
  }

      @override
          Widget build(BuildContext context) {
            super.build(context);
            SystemChrome.setPreferredOrientations([
              DeviceOrientation.portraitUp,
              DeviceOrientation.portraitDown,
            ]);
               ------------
           }

    Widget populateHotProduct() {
    return FutureBuilder<List<HotProducts>>(
      future:this._loadHotProducts,
      builder: (context, snapshot) {
        if (snapshot.connectionState == ConnectionState.waiting &&
            snapshot.hasError == false) {
          return Center(child: CircularProgressIndicator());
        } else if (snapshot.connectionState == ConnectionState.waiting &&
            snapshot.hasError == true) {
          return error_();
        } else if (snapshot.connectionState == ConnectionState.done &&
            snapshot.hasError == true) {
          return error_();
        } else if(snapshot.hasError == true){
          return error_();
        } else{
          return HotProductsLists(hotlist: snapshot.data);
        }
      },
    );
  }
 Future<List<HotProducts>> fetchHotProducts(http.Client client) async {
    final response = await http.post(Configuration.url + "api/getHotProducts");
    if (response.statusCode < 200 || response.statusCode > 300) {
      throw new Exception('Failed to fetch data');
    } else {

      return compute(parseHotProducts, response.body);
    }
  }

  static List<HotProducts> parseHotProducts(String responseBody) {
    final parsed = json.decode(responseBody).cast<Map<String, dynamic>>();
    return parsed.map<HotProducts>((json) => HotProducts.fromJson(json)).toList();
  }

One thing, when i try to open another screen with
 Navigator.push(context, new MaterialPageRoute(
                builder:  (BuildContext context) {
                 return  ProductDetailPage(data: newlist[index]);
                }
));

and going back to tab 1, my future is not running again.
So how can i fix my problem, thanks in advance and sorry for my english
Here is my main_page which handle the tab
class MainPage extends StatefulWidget {
  @override
  _MainPageState createState() => new _MainPageState();
}

class _MainPageState extends State<MainPage> {
  bool _isInAsyncCall = false;
  int currentPage = 0;
  GlobalKey bottomNavigationKey = GlobalKey();

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        primaryColor: Configuration.PrimaryColor,
        fontFamily: "Lato",
      ),
      home: Scaffold(
        appBar: AppBar(title: Text("LF Member"), actions: <Widget>[
          IconButton(
            icon: Icon(Icons.notifications),
            onPressed: () {},
          ),
        ]),
        body: Container(
          decoration:
              BoxDecoration(color: Configuration.PrimaryColor.withOpacity(0.2)),
          child: _getPage(currentPage),
        ),
        bottomNavigationBar: FancyBottomNavigation(
          tabs: [
            TabData(
                iconData: Icons.home,
                title: "Home",
                onclick: () {
                  final FancyBottomNavigationState fState =
                      bottomNavigationKey.currentState;
                  fState.setPage(2);
                }),
            TabData(iconData: Icons.list, title: "Products"),
            TabData(iconData: Icons.local_mall, title: "Store"),
            TabData(iconData: Icons.person, title: "Profile")
          ],
          initialSelection: 0,
          key: bottomNavigationKey,
          onTabChangedListener: (position) {
            setState(() {
              currentPage = position;
            });
          },
        ),
      ),
    );
  }

  _getPage(int page) {
    switch (page) {
      case 0:
        return DashboardPage();
      case 1:
        return ProductCategoryPage();
      case 2:
        return StorePage();
      case 3:
        return ProfilePage();
      default:
        return Column(
          mainAxisSize: MainAxisSize.min,
          children: <Widget>[
            Text("This is the basket page"),
            RaisedButton(
              child: Text(
                "Start new page",
                style: TextStyle(color: Colors.white),
              ),
              color: Theme.of(context).primaryColor,
              onPressed: () {},
            )
          ],
        );
    }
  }
}

So, you are doing
@override
  void initState() {
    _loadHotProducts = fetchHotProducts(http.Client());
    super.initState();
  }```

That means that, (iusing your last example first):
- You open the page, the `initState` is called
- You navigate to a new page with `push`
- You return to the previous page, since it’s not a new page, the `initState` is not called

As for the tabs, the probable cause for this is that you are creating a `new` instance of the page every time you change tabs. So, this means that every time you switch to tab A, the page is created and `initState` is called again, creating a new `Future`.

To fix this, you can have all your pages inside a list (created in the `initState` of the parent widget), and when you are switching tabs you get the page from that list. This way you can preserve the widget without recreating it every time you change tabs.

Hi @bobykurniawan11
Does this solution fix your issue?
Thank you

Hi @TahaTesser , @Vanethos's  answer help me to understand. I use BLoC to fix this problem.


SUTD Archery - Induction Course

Session Name
SUTD Archery - Induction Course
Abstract
Attending this induction course is necessary for students to participate in our regular weekly sessions. They will be introduced to some basic archery theory, the safety aspects of shooting and shooting range procedures.
Student Organisation organising this session
SUTD Archery
Event Format
Physical
Session intent (what to expect)
Participants will not require any prior knowledge of archery. Participants of this induction course will be allowed to attend future regular archery sessions. The course will focus on some basic archery theory, the safety aspects of shooting and shooting range procedures.
Session details
Date and Time: Friday, 1 Oct, 3-5pm
Venue: MPH (Have already checked and booked with Yuxin on this date and time)
Number of participants will have to depend on the maximum allowed by OSL after P2HA. Based on latest advisory before P2HA from OSL on 18 Jul, Archery was allowed to continue in groups of 5, up to 30 pax or venue capacity.
Workshop facilitators
Tan Le Xuan - President
Bridget Chua - Vice-President
Muhammad Hazmi - Captain
Ivan Tandyajaya - Logistics
Eliana Setiabudi - Publicity
Sign up link
https://forms.office.com/r/WpfytwkM7L
Logistical details / Budget Needed
$100 - Prizes (Archery equipment items)
Club Email<EMAIL_ADDRESS>

Good day, unfortunately, the date of the session falls on the 1 October which is out of the DiscoverSUTD timeframe, hence we will not be able to support funding for your session.

HI @makehahaz, thanks for clarifying, would just like to check: If we do not need funding, are we still able to tag this event under DiscoverSUTD, since it's just 1 day after the official end?

Hi @Sorahawk , just checked with the team that anything out of the 30 Sep range won't be tagged under DiscoverSUTD as the timeframe was given to all fifth rows.
Hope your club understands that this is part of being fair to other fifth rows as well!

Hi @makehahaz, we understand thanks for clarifying! I will close this issue since we won't be hosting the event under DiscoverSUTD


any plan for button groups?

just like the twitter bootstrap button groups

I was looking for the same thing and threw this together last night -- it covers the simple case:
https://jsfiddle.net/AJLeonardi/2y70dj8h/4/


Fix bug #760

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.zhangpan seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.zhangpan seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Expose the information about the order of pending workloads

What type of PR is this?
/kind feature
/kind api-change

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #168
Special notes for your reviewer:
Does this PR introduce a user-facing change?

NONE

cc @trasc

Any additional comments @tenzen-y?
/approve

I've added another risk of many requests to update LocalQueue statuses after workload admission. PTAL

@mwielgus @alculquicondor I think I have addressed all the comments. Let me know if there is something more to be done or clarified.

LGTM from my side.

/approve
I'll leave the lgtm to @mwielgus

/hold

/remove-hold


description for "str" has an extra space

The description is:
"...display the internal str ucture of an R object" - but it should be "structure"

The original doc text for str bolds the "str" in "structure" using <b> tags. It looks like we inadvertently insert a space there when stripping tags.

https://github.com/Microsoft/RTVS/commit/16bbfb33f3c03917ba44003ed507f46f3920cfac


Disable colours when the output is not a terminal

I was watching a beanstalk tube by running watch ./beanstool stats and the colorised output was leading to output like this: | ^[0;32m469     ^[0m |.
This patch fixes that.

Awesome, thanks! When I run ẁatch, I use it with watch --color ;)

Thanks for the tip! I'll remember that in future.


[DOCFIX] Add newlines at the end of file

Add newlines to some documentation.

alluxio-bot, merge this please!


racket-mode: avoid using flycheck-disable-checker

The problem is that at this point, flycheck-mode might not be active
yet. Calling flycheck-disable-checker will always call (flycheck-buffer)
which would result in an error: (user-error "Flycheck mode disabled")

Thanks!


Revert "Bower"

Reverts GoogleChrome/chromedeveditor#4067

(e.g. I signed it!)
On 6/16/16, googlebot<EMAIL_ADDRESS>wrote:

Thanks for your pull request.  It looks like this may be your first
contribution to a Google open source project. Before we can look at your
pull request, you'll need to sign a Contributor License Agreement (CLA).
:memo: Please visit https://cla.developers.google.com/ to sign.
Once you've signed, please reply here (e.g. I signed it!) and we'll
verify.  Thanks.

If you've already signed a CLA, it's possible we don't have your GitHub
username or you're using a different email address.  Check your existing
CLA data and verify that your
email is set on your git
commits.
If you signed the CLA as a corporation, please let us know the company's
name.

You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/GoogleChrome/chromedeveditor/pull/4074#issuecomment-226409967

<EMAIL_ADDRESS>

chrome.tabs.query({'active': true}, function(tabs) {
chrome.tabs.update(tabs[0].id, {url: newUrl});
});
someOtherFunction();


how can I customize my own template for ionic generate

how can I customize my own template for ionic generate?

Hello! Thanks for opening an issue with us! As this seems like more of a support question i will urge that you ask this question on our forum or on our slack channel. Thanks for using Ionic!


"(Click me)" opens with a new tab

Currently, "(Click me)" link has potentially lose your work by accidently clicking it.
By opening in a new tab, this can be prevended.

Good point - you changed the format in the PR though, so I just went ahead and made my own commit to add the same without altering the format, and applied it to the other link.
Thank you!


For ComfyUI_Dave_CustomNode, the content is not displayed

The contents of the extension in swarmui are not displayed, the options are not displayed, and the right-click menu is not displayed.
I don't know if I should direct this question to the plugin author or swarmui.
Here is a comparison of the comfyui and swarmui interfaces，What should be done about this problem
comfyui：

swarmui：

That'd be an issue with the relevant extension to fix - I tried googling the name, and only found this https://github.com/Davemane42/ComfyUI_Dave_CustomNode which was archived 8 months ago?


Aura coordenates

the radius of the aura is doing a strange calculation.
my aura has a 5 radius. (in Large creature)
It was supposed to take only tokens within the radius of the large creature.
but when it is on the left and at the top, it acts right (not catching) but if it is on the right or below, the effect catches the token.

large creature measuring is an anomoly that im not quite sure how to fix. The system seems to pick a random square to measure from, and use that staticly.


📝 Remove highlights in indexes.md

The full file preview doesn't have highlights normally, so I think this is a mistake :thinking:

Yep, good point, thanks! :rocket:


[HDInsight] Add encryptionAtHost property

MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[ ] I commit to follow the Breaking Change Policy of “no breaking changes
[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

[ ] Ensure to check this box if one of the following scenarios meet updates in the PR, so that label “WaitForARMFeedback” will be added automatically to involve ARM API Review. Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs, all “removals” and “adding a new property” no more require ARM API review.

Adding new API(s)
Adding a new API version
Adding a new service

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If there are following updates in the PR, ensure to request an approval from API Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in stable version
[ ] Removing properties in stable version
[ ] Removing API version(s) in stable version
[ ] Updating API in stable version with Breaking Change Validation errors
[ ] Updating API(s) in preview over 1 year

Please follow the link to find more details on PR review process.

/azp run automation - sdk

Can one of the admins verify this patch?

@aim-for-better 1041 - AddedPropertyInResponse is considered as breaking change. why do you need add such property? also please clarify if there's customer impact.

@aim-for-better 1041 - AddedPropertyInResponse is considered as breaking change. why do you need add such property? also please clarify if there's customer impact.

Hi @leni-msft , I added this property to support a new feature. Customer can specify it to true to enable encryption at host feature.  This change won't have impact on old cluster so there isn't customer impact.


fix!: rename some package names

Close #19
Close #3
Target repos:

Non-default branch repos
Forks
Inappropriate repo names (owner/neovim, owner/nvim, etc.)

Spec change:

Full format is given by type:owner/repo:branch:rename.
branch can be empty, so owner/repo::rename uses default branch
but renamed to rename.

Results of bin/update-vim-plugins awesome must be adjusted by these renamers.


Deobfuscator renames com.* packages to p027com.*, but not all

Hi,
I've noticed in Bankin app (https://drive.google.com/open?id=1RhnwzWEI2uaTFbfA5BMTUG5GlIQ969rc) that when I supply option --deobf and open sources - com, I see only google package, but all other packages starting with com were moved to p027com. It seems to be an error.
Moreover, I will apply a patch which changes class names in AndroidManifest.xml to new ones from the deobfuscation map.

Is this still a issue? If you don't like com being rename just set --deobf-min to 3.

@skylot
Thanks, but why it doesn’t rename all com packages in this case? Is it a bug?

Ok. I found source of this issue :(
Jadx has class name checker (in RenameVisitor) it renames illegal java names (like starts with number or dollar sign) and executed independently of deobfuscator and even deobfuscation is disabled. So bug was in renamer it don't use pkg alias and overwrites class pkg name.
I don't like this approach were we have two places of renaming, it must be merged.
Anyway I fixed this issue. You can check it.

Thanks, seems to be fixed!


Addtravellingsalesmanproblem

will this be merged?

added travelling salesman problem . please check and merge it.(pr#157)

@vashuteotia123 please check and merge it with a label of  hacktoberfest


Rollover text in tutorial box

Is your feature request related to a problem? Please describe.
Sometimes there's a word we use that's tech-speaky enough to want to make sure students know what it is, even if we think it's already been made clear in another place.  For example, we might want to use the word "sprite" in a tutorial for brand-new users, but we don't want to take the time to give a full sentence description of what the word means.  This is especially important in early stages when we might be presenting 3-4 new words in one chunk of instructions.
Describe the solution you'd like
It would be great if students could roll over the word with their mouse and get a mouseover text type of box so they could decide how much refreshing they need.
Describe alternatives you've considered
I'm open to hear more.
Additional context
Eventually, it would be great if we could nestle other cumbersome features inside the instructions.  Want someone to create a tilemap and make it so they can't go through certain areas? Don't waste two paragraphs, have a popout for things we find ourselves re-teaching over and over again!

Turns out, we can do this!


paper for implementation of blackbox attribute inference attack

Hi,
Thank you for the implementation of the toolbox.
I am interested in using black-box attribute inference attack model blackbox model . But I have difficulty in understanding how it works and how to use the implementation in my case. I am wondering if this just attribute inference attack or model inversion attribute inference attack since it makes use of predicted labels from a model and values (marginals).
Could you please share with me the reference?
looking forward to hearing back from you.
Thank you.
Bests,
Manel.

Hi @SlokomManel Thank you for using ART! @abigailgold What do you think?

Hi @SlokomManel I'm not sure I understand the distinction you're making between attribute inference attack and model inversion attribute inference attack. The idea is to use n-1 features along with the model's output to predict the value of a missing feature. It is the same kind of attack as in https://dl.acm.org/doi/10.1145/2810103.2813677 but with a different implementation which is more similar to the black-box inference attack of Shokri et al (https://arxiv.org/abs/1610.05820).
If you have any specific questions about how to use it you're welcome to ask. There is also a notebook available: https://github.com/Trusted-AI/adversarial-robustness-toolbox/blob/main/notebooks/attack_attribute_inference.ipynb
Hope this helps.

Hi,
Thank you so much for your reply.
I am actually looking at the code of blackbox attribute inference in https://github.com/Trusted-AI/adversarial-robustness-toolbox/blob/main/art/attacks/inference/attribute_inference/black_box.py .
In line 327, we have "predictions = np.array([self._values[np.argmax(arr)] for arr in predictions])".
Values is a vector with target attribute (to be attacked) classes sorted in ascending order. Predictions are the output predictions of the classifier.
I am not able to get what sort of aggregation or combination function that is used here. It looks like the success of inference here heavily depends on the "values" vector. I was not able to find in literature how this combination of values vector and predictions vector is done.
Thank you,
Bests.

In this line, predictions is actually the output of the attack. So these are the attack model's predicted values for the attacked feature (not the predictions of the original/attacked/target model).
The values vector is just used to translate those predictions back to the original values (if they are different from just 0, 1, 2, etc.). So as a simple example, let's assume the attacked feature has three possible values: -1, 0 and 1. The attack model will output those as 0, 1, 2 (sequential labels starting at 0). But if you supply a values vector of [-1, 0, 1], then in the output of the attack, 0 will be translated to -1, 1 to 0 and 2 to 1 so that you get back values in the original domain of the feature.

Ascending order of the values themselves. Not their frequency.

Right. So if we go back to this example :
values = [-1, 0, 1] # ascending order
predictions = [0,0,1]
This means that the final prediction will be class "1".
But if values = [0, -1, 1], the final prediction will change. So there is a sort of link/connection between values and prediction. At the end, let's assume we have three categories in the target attribute: c1, c2, c3.  if there are the same naming of classes in the prediction vector returned by the attacker classifier and values vector. But the only difference is that values vector has a different order. Classifier is trained to say for a specific user, the inferred class is c2 but if we use values vector it becomes c3.
Sorry that i am trying to reformulate this multiple times in different ways but my main concern is about "the shift in prediction of classifier". Does it make sense to you?

If you look at the code in the fit() method, the attacked feature is always one-hot-encoded after applying one of the methods float_to_categorical or floats_to_one_hot. Both of these methods sort the unique values in the data in ascending order before one-hot encoding them. So if 'values' is always sent in ascending order of the values, these two will match and you will get the correct prediction. This is exactly why it is required to send them in ascending order.


[RBAC] More settings for disabling certain features

Migrating some items from https://github.com/ansible/django-ansible-base/issues/124 and adding a few new things

something to prevent creating custom roles for the team model, as in, the RoleDefinition.content_type is team
flag to prevent creating custom organization roles with the member_team permission?
flag to prevent giving teams a role with "member_team" permission
setting that will completely disable creating of custom roles via the API

https://github.com/ansible/django-ansible-base/pull/45#discussion_r1499874218

Can you document what all of these settings do? Settings like ANSIBLE_BASE_TEAM_TEAM_ALLOWED are very confusing.

This setting is an existing enable/disable setting. Quote from @newswangerd comment.
Somehow, exactly how I don't care, we should collect all the RBAC enable/disable setting flags and document them in a single place.

The collection of enablement settings is now:
    # Specific feature enablement bits
    # For assignments
    ANSIBLE_BASE_ALLOW_TEAM_PARENTS = True
    ANSIBLE_BASE_ALLOW_TEAM_ORG_PERMS = True
    ANSIBLE_BASE_ALLOW_TEAM_ORG_ADMIN = True
    # For role definitions
    ANSIBLE_BASE_ALLOW_CUSTOM_ROLES = True
    ANSIBLE_BASE_ALLOW_CUSTOM_TEAM_ROLES = False
    # Allows managing singleton permissions
    ANSIBLE_BASE_ALLOW_SINGLETON_USER_ROLES = False
    ANSIBLE_BASE_ALLOW_SINGLETON_TEAM_ROLES = False

Pushed a commit, closing as done.


Add LogImplementation.logException and better error types for OpenAI

This adds:

A LogImplementation.logException method, which by default invokes LogImplementation.log. In the default implementation an exception is logged with level "error" the first time it is seen, and level "trace" subsequently (as it propagates up the element tree).
More consistent error handling for OpenAI HTTP exceptions. We now parse the error message out of the response body and put it in the Error message.

We can ignore the Vercel warnings.

When I run this against simple-chat with an example that overflows the token count, I get:
OpenAIError: OpenAI createChatCompletion request failed with status code 400: This model's maximum context length is 4097 tokens. However, your messages resulted in 12519 tokens. Please reduce the length of the messages.
    at checkOpenAIResponse (file:///Users/nth/code/ai-jsx/packages/ai-jsx/dist/esm/lib/openai.js:88:15)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async OpenAIChatModel (file:///Users/nth/code/ai-jsx/packages/ai-jsx/dist/esm/lib/openai.js:163:5) {
  statusCode: 400,
  responseBody: '{\n' +
    '  "error": {\n' +
    `    "message": "This model's maximum context length is 4097 tokens. However, your messages resulted in 12519 tokens. Please reduce the length of the messages.",\n` +
    '    "type": "invalid_request_error",\n' +
    '    "param": "messages",\n' +
    '    "code": "context_length_exceeded"\n' +
    '  }\n' +
    '}\n',
  responseHeaders: {
    date: 'Tue, 13 Jun 2023 02:09:18 GMT',
    'content-type': 'application/json',
    'content-length': '282',
    connection: 'close',
    'access-control-allow-origin': '*',
    'openai-organization': 'fixie-ai',
    'openai-processing-ms': '5198',
    'openai-version': '2020-10-01',
    'strict-transport-security': 'max-age=15724800; includeSubDomains',
    'x-ratelimit-limit-requests': '3500',
    'x-ratelimit-limit-tokens': '90000',
    'x-ratelimit-remaining-requests': '3499',
    'x-ratelimit-remaining-tokens': '85903',
    'x-ratelimit-reset-requests': '17ms',
    'x-ratelimit-reset-tokens': '2.73s',
    'x-request-id': '6bca147a29a141d30e86aa8eec30b143',
    'cf-cache-status': 'DYNAMIC',
    server: 'cloudflare',
    'cf-ray': '7d66ca49faec8f17-IAD',
    'alt-svc': 'h3=":443"; ma=86400'
  },
  errorResponse: {
    error: {
      message: "This model's maximum context length is 4097 tokens. However, your messages resulted in 12519 tokens. Please reduce the length of the messages.",
      type: 'invalid_request_error',
      param: 'messages',
      code: 'context_length_exceeded'
    }
  }
}

This is a big improvement over main's current behavior, which is:
file:///Users/nth/code/ai-jsx/packages/ai-jsx/dist/esm/lib/openai.js:147
        throw new Error(`OpenAI request failed with status code ${chatResponse.status}`);
              ^

Error: OpenAI request failed with status code 400
    at OpenAIChatModel (file:///Users/nth/code/ai-jsx/packages/ai-jsx/dist/esm/lib/openai.js:147:15)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)

Node.js v18.16.0

That said, I would like to present a more polished dev experience. Something that looks more like:
----------------------------------
| AI.JSX(1234): Model Call Error  |
----------------------------------
This model's maximum context length is 4097 tokens. However, your messages resulted in 12519 tokens. Please reduce the length of the messages.

For more detail, see ai-jsx.log.
Need help? https://ai-jsx-docs.vercel.app/

In that last part, we have a common suffix.
This can be a separate PR.


C++: Use optimistic allocation instead of Arrow Builders

Instead of using the Arrow builders, optimistically allocate buffers to avoid unncessary memcpys. This should bring us a significant performance improvement.
Reporter: Uwe Korn / @xhochy
Assignee: Uwe Korn / @xhochy
Note: This issue was originally created as PARQUET-762. Please see the migration documentation for further details.

Uwe Korn / @xhochy:
PR: https://github.com/apache/parquet-cpp/pull/183

Wes McKinney / @wesm:
Issue resolved by pull request 183
https://github.com/apache/parquet-cpp/pull/183


Some examples:

An exception is returned if the parameter doesn't exist

Awesome! I don't think that we need to show fa_icon in the terminal (that's just the name of the icon for the web laucher), but otherwise looks brilliant 🚀

Thank you @ewels!
Could we merge this PR and #6 if you agree?
Ping @pditommaso for a review too :)

@mirpedrol do you not have permissions to merge PRs yourself? I don't have access to change this, but maybe @pditommaso can upgrade you..

do you not have permissions to merge PRs yourself?

No, I don't see the merge button

Ok, I'm not sure how busy @pditommaso is so I'll merge. Feel free to review post-merge @p!


PCR-304,362

PCR 304 and PCR 362 Commit and Acknowledgment fix

Will do changes in map fn

PR updated, map fn changed


feat: add GitHub Actions for hotfix and release workflows, including version bumping and publishing to NPM

@nico-ptrs Yes, but for public projects, actions can't be used from our venly-github-workflows.

@nico-ptrs Yes, but for public projects, actions can't be used from our venly-github-workflows.


Recognize *.BUILD files

(sorry about the weird old title)
I noticed that *.BUILD files are not recognized by this plugin as being BUILD files. Could you please add support for recognizing these files as being of the bzl filetype? It's very useful for formatting new_http_archive and new_git_repository specified build files. For example:
https://github.com/tensorflow/tensorflow/blob/master/third_party/jpeg/jpeg.BUILD

Easy enough to add of course, but do you happen to know the expected scheme for BUILD file naming? I looked around for it on https://bazel.build/versions/master/docs/build-ref.html but couldn't find anything definitive.

(Might as well add any other known BUILD patterns).
Also, for reference this is an offshoot from https://github.com/google/vim-codefmt/issues/82

Unfortunately I don't know, but from looking at Bazel's and TensorFlow's projects it looks like they use *.BUILD and also BUILD.*:
https://github.com/bazelbuild/bazel/blob/master/tools/cpp/BUILD.tpl
So it's a good start.

I've proposed a pull request for this in #12 .

Fixed by #12.


docs: adding documentation for regressors

-- Add documenting for regressors:
FIMTDD
ORTO
KNNRegressor
Passive Aggressive Regressor
SGD Regressor
SOKNL
SOKNLBT
-- Update __init__.py accordingly

@hmgomes, amendments have been implemented, please take a look.

thanks @YibinSun , it is looking much better now


Fixed to handle undefined field when grouping

Fix: Commit # 24155ad breaks group by field if the field is missing in any of the row. This is because of val.toString() which doesn't work on undefined.
http://plnkr.co/edit/9TaFbiAIsVSAObPpcgO4?p=preview
Should be changed to:
val = (val === "" || val === null || val === undefined) ? 'null' : val.toString();

Thanks


Module parse failed: Unexpected token (1:0)

Sorry to dig up this thread again. I am also facing a similar issue
    ERROR in ./node_modules/vue-bootstrap-typeahead/src/components/VueBootstrapTypeahead.vue 1:0
    Module parse failed: Unexpected token (1:0)
    You may need an appropriate loader to handle this file type.
    > <template>
    |   <div>
    |     <div :class="sizeClasses">
     @ ./ClientApp/views/home.vue?vue&type=script&lang=js& (./node_modules/babel-loader/lib!./node_modules/vue-loader/lib??vue-loader-options!./ClientApp/views/home.vue?vue&type=script&lang=js&) 2:0-97 35:27-48
     @ ./ClientApp/views/home.vue?vue&type=script&lang=js&
     @ ./ClientApp/views/home.vue
     @ ./ClientApp/router/routes.js
     @ ./ClientApp/router/index.js
     @ ./ClientApp/boot-app.js

I tried to provide full import path like @HasnatH 's but it still fails.
I am running
"vue": "2.5.16",
"vue-loader": "15.4.2",
"webpack": "^4.28.2",
"webpack-cli": "3.1.2",
"bootstrap-vue": "^2.0.0-rc.11",
"vue-bootstrap-typeahead": "^0.2.6",
"@babel/core": "7.2.2",
"@babel/plugin-syntax-dynamic-import": "^7.2.0",
"@babel/plugin-transform-runtime": "7.2.0",
"@babel/preset-env": "7.2.3",
"@babel/runtime": "7.2.0",
similar to https://github.com/alexurquhart/vue-bootstrap-typeahead/issues/12

+1
15 Mar 12:56:08 - error: undefined of node_modules/vue-bootstrap-typeahead/src/components/VueBootstrapTypeahead.vue failed. rror: Unexpected token (5:4) ^G

Had the same issue. My problem was related to my webpack vendor config. In my case I needed to add a .vue$ test referencing vue-loader that included node_modules/vue-bootstrap-typeahead to my vendor configuration (which previously didn't have a test for .vue files - my primary webpack config excluded node_modules). Everything seems to work fine after that.

Same here, how to parse this module so a standard nuxtjs config can import it as a plugin ?

Same here. Found a solution following the official documentation of nuxt : https://nuxtjs.org/guide/plugins
Create a nuxt plugin and don't forget to set build.transpile property in the nuxt.config according to the documentation.


gethostname should return name instead of ip address again

Same here, public addresses works fine but not local.

Hi, this has to do with the DHCP and DNS server on your local router, is there any other way to get the device name, so its not depended on the router configs.


Bump build to catch up to PDAL 1.9.1

as the title says

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.


Issues loading 10x data

❓ Questions and Help
We have a set of listed tutorials available on the website.
Hello, I am trying to load 10x data into R using repLoad and am getting the following errors. Am I using the correct 10x file? Any ideas for how to resolve my issue?
> immdata <- repLoad("filtered_contig_annotations.csv",.format = "10x")
== Step 1/3: loading repertoire files... ==
Processing "<initial>" ...
  -- Parsing "filtered_contig_annotations.csv" -- 10x
unknown format, skipping

== Step 2/3: checking metadata files and merging... ==
Processing "<initial>" ...
  -- Metadata file not found; creating a dummy metadata...
Dropping  1 column(s) from .metadata.txt. Do you have spaces or tabs after the name of the last column? Remove them to ensure everything works correctly.

== Step 3/3: splitting data by barcodes and chain types... ==
Done!
> str(immdata)
List of 2
 $ data: Named list()
 $ meta: tibble [0 × 0] (S3: tbl_df/tbl/data.frame)
 Named list()
>

Hi @lothm2014
Thank you for using the package and helping the community!
You definitely use the correct file. However, you shouldn't pass the .format argument as immunarch automatically detects the file format. Can you please remove it and see if it works?
immdata <- repLoad("filtered_contig_annotations.csv")

Thanks for the quick response. That was successful. I did just want to pass along the following warnings when loading the data:
immdata<-repLoad(("filtered_contig_annotations.csv"))
== Step 1/3: loading repertoire files... ==
Processing "<initial>" ...
  -- Parsing "filtered_contig_annotations.csv" -- 10x (filt.contigs)
Warning: 151 parsing failures.
 row             col           expected               actual                              file
1041 NA              18 columns         27 columns           'filtered_contig_annotations.csv'
1042 is_cell         1/0/T/F/TRUE/FALSE TTRBV7-2             'filtered_contig_annotations.csv'
1042 high_confidence 1/0/T/F/TRUE/FALSE TRBJ1GACTrTCTGATTG-1 'filtered_contig_annotations.csv'
1042 length          a double           TrTCTGATTG-1         'filtered_contig_annotations.csv'
1042 full_length     1/0/T/F/TRUE/FALSE TRBV5-6              'filtered_contig_annotations.csv'
.... ............... .................. .................... .................................
See problems(...) for more details.

  [!] Removed 277 clonotypes with no nucleotide and amino acid CDR3 sequence.

== Step 2/3: checking metadata files and merging... ==
Processing "<initial>" ...
  -- Metadata file not found; creating a dummy metadata...

== Step 3/3: splitting data by barcodes and chain types... ==
Splitting repertoires by their chain type. Columns Chain and Source added to the metadata.
  -- Column Chain specifies chain types: TRA, TRB, etc.
  -- Column Source specifies the initial sample name for a repertoire after splitting by chain types
Done!
Warning message:
The following named parsers don't match the column names: barcode,is_cell,contig_id,high_confidence,length,chain,v_gene,d_gene,j_gene,c_gene,full_length,productive,cdr3,cdr3_nt,reads,umis,raw_clonotype_id,raw_consensus_id

Thank you! Yes, we plan to fix the warnings, but not at the moment, I'm sorry. However, we would be glad to help anyone who seeks to fix them! Immunarch is finally on GitHub so we can work with the RepSeq / single-cell community even closer.


Mvn delete

Hi,
I propose this PR to close #277
It adds two maven services

delete existing version property using its identifier from maven pom
delete dependency from pom using its package and artifacts id. May content other fields such as scope

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Codecov Report

Merging #318 (47c36bf) into main (deb0d8b) will not change coverage.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##                main      #318   +/-   ##
===========================================
  Coverage     100.00%   100.00%
- Complexity       456       461    +5
===========================================
  Files             80        80
  Lines           1453      1476   +23
  Branches          32        32
===========================================
+ Hits            1453      1476   +23

Impacted Files
Coverage Δ

...ool/maven/application/MavenApplicationService.java
100.00% <100.00%> (ø)

...tor/buildtool/maven/domain/MavenDomainService.java
100.00% <100.00%> (ø)

...frastructure/secondary/ProjectLocalRepository.java
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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

Thanks a lot @MSaguer for your contribution. And congrats for keeping the coverage at the max :-)
I added a minor comment. What do you think?


feat: support optional chaining in ?module mode

As per the discussion we had on twitter here, this adds [@babel/plugin-proposal-optional-chaining] to allow the ?module mode to support optional chaining.
Happy new year 🍾🙂

Yeah we would like to have this available as well. We tried to transpile out optional chaining on prepublish, but this was met with a ton of problems with regards to types / TSC and we would have to force our TS users to set skipLibCheck: true in their configs. https://github.com/ing-bank/lion/issues/1169#issuecomment-772440321 for reference

@mjackson is there anything we need to get this in? We're about to publish lit-html 2.0 which uses ES2020. We'd love to get this and nullish-coalescing in.
@thepassle would you want to add nullish-coalescing to this PR too?

@thepassle would you want to add nullish-coalescing to this PR too?

Added 👍

Thanks for the PR, @thepassle! My apologies this took so long to merge.

Thanks @thepassle and @mjackson !

I forgot to ask: what URLs were you trying to use this with @justinfagnani / @thepassle? I'd like to write a quick test for it before I deploy.

Ah, I think I found one: /playground-elements@0.3.6/playground-ide.js?module (from this tweet).
I deployed this PR to the staging server, unpkg.dev. This should be working as expected: https://unpkg.dev/playground-elements@0.3.6/playground-ide.js?module


Feature Request: Snipptes integration

Integrate also Snippets by Bitbucket
https://bitbucket.org/dashboard/snippets

@tschinz Thanks for the suggestion :) Since I'm not using Bitbucket, I don't have a plan to integrate its snippet system.  Luckily, Lepton's snippet APIs are encapsulated in an independent module which means we could potentially integrate Bitbucket's snippet APIs by swapping this module.
Feel free to let me know you are interested and I'm more than happy to give you a code tour.

Thanks for the answer. Good to know for the encapsulation of API. I'll have a look but I think my knowledge in this field of programming is too much limited.
Thanks anyway for an awesome tool, it is in use everyday ;-)


[BUG] Missing functionality - Service offerings > VPC offerings - change order

Describe the bug
In the old UI there is the option to change the order in 'Template > ISO' page, which is missing in Primate UI.
To Reproduce
Steps to reproduce the behavior:

Go to 'Service offerings > VPC offerings' and look at the grid - no way to change the list order.

Expected behavior
There must be an option to change the order of the items, as in the old UI.
Screenshots
Old UI:

New UI:

Desktop (please complete the following information):

OS: Linux Ubuntu 18.04 LTS
Browser: Chrome
Version: 80.0.3987.132 (Official Build) (64-bit)

Additional context
None

Fixed now https://github.com/apache/cloudstack-primate/commit/c9c7cc3cf40e88c631252cc1971f1c10516989a0

@vladimirpetrov please review latest master

Fix confirmed, closing the issue.


Add internal chip

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

13 of 13 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 99.679%

Totals

Change from base Build<PHONE_NUMBER>2:
0.0%

Covered Lines:
13066

Relevant Lines:
13067

💛  - Coveralls


chore(gitignore): ignore globals.js

Hm, my setup isn't generating a globals.js.

The tsconfig.json should prevent outputting globals.js.

After deleting globals.js, it appears to be gone forever.  Sorry!


ValueError: malformed node or string: None

I get this error on the ---> 21 API_KEYS = literal_eval(os.getenv("API_KEYS")) line and I don't understand why

you have to add .env file your project directory.
Sample Env file:
API_KEYS=["<api_key_1>", "<api_key_2>"] # To fetch the news feed
GITHUB_API_TOKEN=<github_token> # To commit the news feed on github


PR: Avoid that code analysis tree collapses on selection

Description of Changes

[ ] Wrote at least one-line docstrings (for any new functions)
[ ] Added unit test(s) covering the changes (if testable)

[x] Included a screenshot or animation (if affecting the UI, see Licecap)

In the current master, the tree with code analysis issues collapses on selection:

After this PR, no collapse:

Issue(s) Resolved

It does not solve #9564 but at least mitigates one of the issues mentioned there.
Affirmation
By submitting this Pull Request or typing my (user)name below,
I affirm the Developer Certificate of Origin
with respect to all commits and content included in this PR,
and understand I am releasing the same under Spyder's MIT (Expat) license.

Oscar Gustafsson/oscargus

After checking the test reports, I must say that it is not really clear how my changes caused failing tests all over... I would even be tempted to say that I doubt the code change caused them...

I would even be tempted to say that I doubt the code change caused them...

@oscargus I think you are right, my PR fails with the same error.

@oscargus, please rebase or merge with master to fix the errors in our tests.

Thanks! Still some errors which I think is unrelated:
Travis 2.7 and 3.7 with conda
spyder/app/tests/test_mainwindow.py::test_lock_action
No output has been received in the last 10m0s, this potentially indicates a stalled build or something wrong with the build itself.

Travis 3.6 with conda
Traceback (most recent call last):
  File "/home/travis/build/spyder-ide/spyder/spyder/plugins/editor/lsp/manager.py", line 244, in send_request
    client.perform_request(request, params)
  File "/home/travis/build/spyder-ide/spyder/spyder/plugins/editor/lsp/client.py", line 327, in perform_request
    _id = handler(params)
  File "/home/travis/build/spyder-ide/spyder/spyder/plugins/editor/lsp/decorators.py", line 42, in wrapper
    _id = self.send(method, params, kind)
  File "/home/travis/build/spyder-ide/spyder/spyder/plugins/editor/lsp/client.py", line 270, in send
    self.zmq_out_socket.send_pyobj(msg)
  File "/home/travis/miniconda/envs/test/lib/python3.6/site-packages/zmq/sugar/socket.py", line 606, in send_pyobj
    return self.send(msg, flags=flags, **kwargs)
  File "/home/travis/miniconda/envs/test/lib/python3.6/site-packages/zmq/sugar/socket.py", line 395, in send
    return super(Socket, self).send(data, flags=flags, copy=copy, track=track)
  File "zmq/backend/cython/socket.pyx", line 725, in zmq.backend.cython.socket.Socket.send
  File "zmq/backend/cython/socket.pyx", line 772, in zmq.backend.cython.socket.Socket.send
  File "zmq/backend/cython/socket.pyx", line 247, in zmq.backend.cython.socket._send_copy
  File "zmq/backend/cython/socket.pyx", line 242, in zmq.backend.cython.socket._send_copy
  File "zmq/backend/cython/checkrc.pxd", line 25, in zmq.backend.cython.checkrc._check_rc
zmq.error.ZMQError: Socket operation on non-socket

Same problems on the cancelled and failing Azure pipeline tests, respectively.

Please merge with master to get a new set of fixes for our tests.

Fewer and fewer failures. Still seems to be a problem with ZMQError for Windows tests on Azure pipelines though:
Qt exceptions in virtual methods:
________________________________________________________________________________
Traceback (most recent call last):
  File "D:\a\1\s\spyder\plugins\editor\lsp\manager.py", line 244, in send_request
    client.perform_request(request, params)
  File "D:\a\1\s\spyder\plugins\editor\lsp\client.py", line 327, in perform_request
    _id = handler(params)
  File "D:\a\1\s\spyder\plugins\editor\lsp\decorators.py", line 42, in wrapper
    _id = self.send(method, params, kind)
  File "D:\a\1\s\spyder\plugins\editor\lsp\client.py", line 270, in send
    self.zmq_out_socket.send_pyobj(msg)
  File "C:\Miniconda\envs\test\lib\site-packages\zmq\sugar\socket.py", line 606, in send_pyobj
    return self.send(msg, flags=flags, **kwargs)
  File "C:\Miniconda\envs\test\lib\site-packages\zmq\sugar\socket.py", line 395, in send
    return super(Socket, self).send(data, flags=flags, copy=copy, track=track)
  File "zmq\backend\cython\socket.pyx", line 725, in zmq.backend.cython.socket.Socket.send
  File "zmq\backend\cython\socket.pyx", line 772, in zmq.backend.cython.socket.Socket.send
  File "zmq\backend\cython\socket.pyx", line 247, in zmq.backend.cython.socket._send_copy
  File "zmq\backend\cython\socket.pyx", line 242, in zmq.backend.cython.socket._send_copy
  File "zmq\backend\cython\checkrc.pxd", line 25, in zmq.backend.cython.checkrc._check_rc
zmq.error.ZMQError: Unknown error

(I do not fully understand how some PRs seems to pass without these issues and some not...)

(I do not fully understand how some PRs seems to pass without these issues and some not...)

Unfortunately some tests are very flaky, so they need to be run several times. Please don't worry about them.

Unfortunately some tests are very flaky, so they need to be run several times. Please don't worry about them.

OK, good to know for possibly later contributions!


Added the missing FromMiliseconds Method

Description
Added the method to create timespan from miliseconds.
Motivation and Context

Adding a missing method.
How Has This Been Tested?

Screenshots
Types of changes

[X ] Improvement (non-breaking change that improves a feature, code or algorithm)
[ ] Bug fix (non-breaking change which fixes an issue)
[X ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[X ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[X ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Signed-off-by: GITHUB_USER <GITHUB_USER_EMAIL>

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


accessibility: "More Dates" popup on Course Pages lacks keyboard controls

Steps to Reproduce

go to any product page with multiple runs
Click on the dates to display the pop-up

Expected Behavior

should be able to tab between the dates, because they are links

Actual Behavior

can't tab

Related Issues

#122

Screenshot or Screencast

See https://www.w3.org/WAI/ARIA/apg/example-index/link/link.html for missing code elements/examples.
Need role=link and onkeydown=

@collinpreston I looked into this a while ago but never actually managed to view the popover in a browser. (I was only trying to view it on rc. IIRC, We discovered that in order for "more dates" to show, you need a previous enrollment, which I did not have.)
One thing I noticed in the code is that the contents are not focusable (I believe; like I said, I couldn't actually try it). They are spans but should be anchor tags.
Additionally... I am not sure what the default container for bootstrap popovers is. Appending the popovers to the parent element may help with tab navigation (once the elements are focusable), but possibly could mess up scrolling.

When I looked at it last week, the problem appeared to be the popover is not focusable as Chris mentioned above. I came across this https://getbootstrap.com/docs/4.4/components/popovers/#usage document from bootstrap, It mentioned the tabbing issue for the popover. Not sure if there is a simple solution to it, but I wasn't able to come up with a decent solution with the popover implementation.

@pdpinch, bootstrap recommends using a modal instead of building custom handling to get tab'ing and keyboard accessibility to work with popovers like this.
Reference

In short, simply adding interactive controls to a popover is likely to make these controls unreachable/unusable for keyboard users and users of assistive technologies, or at the very least make for an illogical overall focus order. In these cases, consider using a modal dialog instead.

Should we try our best to make the popover accessible to keyboard users or should we migrate to using a modal?


Old functions can't use imported functions when the script is run a second time.

Problem Description
When the following script is run twice from within spyder, it raises an AttributeError the second time.
Some ideas why this might happen:
I think it is somehow connected to Spyder being written in Pyqt, and code from the first run is affecting the second run, because no new QApplication is created for the second run.
In the second run, the signal "QApplication.lastWindowClosed" is emitted and calling the custom function "log()".
In this function a function from the module logging is used, but Spyder doesn't seem to remember the logging module in this function in the second run, thus "logging" ist set to "None" instead of the imported module when run the second time, resulting in the AttributeError.
In this example I can fix it by disconnecting the "lastWindowClosed" signal everytime at the program start, but in more complicated programms it might take a few more lines.
What steps will reproduce the problem?

create a file pack.py:

# pack.py

import sys
import logging

from PyQt5.QtWidgets import QApplication

def log():
    logging.warning("Test!")

def connect_qapp():
    qapp = QApplication.instance()
    if not qapp:
        qapp = QApplication(sys.argv)
    qapp.lastWindowClosed.connect(log)

Create a file main.py in the same folder:

# main.py

import sys

from PyQt5.QtWidgets import QApplication

from pack import connect_qapp

qapp = QApplication.instance()
if not qapp:
    qapp = QApplication(sys.argv)
connect_qapp()

# the following lines provide a workaround:
"""
try:
    qapp.lastWindowClosed.disconnect()
except TypeError:
    pass
"""

qapp.lastWindowClosed.emit()
qapp.exec_()

Run main.py twice from Spyder

What is the expected output? What do you see instead?
The expected output is no AttributeError, but I see an AttributeError when the programm is
run a second time.
Paste Traceback/Error Below (if applicable)

  File "pack.py", line 10, in log
    logging.warning("Test!")

 We'll take a look at it in our next release.

Is this still the case @impact27?
Thanks!

No, it's not because we run things in an empty namespace by default now.


add flag to each page allowing to hide base page (see issue #537)

Tested on ESP32 with ILI9341 and XPT2046.

Closing PR per request in #537


Enable user-level management of index credentials via uv (& keyring)

Summary
Currently reading credentials from keyring is only supported when a username is provided in the URL of the index. This prohibits to define indexes - in pyproject.toml - that are shared within a team. See these two issues for further details:

https://github.com/astral-sh/uv/issues/8810
https://github.com/astral-sh/uv/issues/9165

In general this PR provides two things:
Setting credentials for an index via CLI
With this MR you can run uv index credentials add --name <name-of-the-index> [--username <username>]
This will ask for the password of the user. A keyring entry will be made with the url of the index, the username and the password. The username and the index will be appended to "<uv_cache_dir>/auth.toml". "auth.toml" has the following structure
[indexes.index1]
username = "my_user1"

[indexes.index2]
username = "my_other_user"

Using credentials
When reading the credentials in uv_distribution_types::index::Index.credentials it's now additionally checked if credentials have been configured via "<uv_cache_dir>/auth.toml" and keyring. The current implementation of reading the credentials from the environment variables UV_INDEX_XXX has priority over keyring authentication.
Test Plan
I have added unit tests to the newly defined uv_auth::keyring_config which takes care of loading, storing and modifying the auth configuration.
I also want to add a test for uv_auth::credentials::Credentials.from_keyring. But am currently struggling with mocking keyring and config file.
I manually tested the new command uv index credentials add --name <name-of-the-index> as I could not find examples for testing commands.
Further remarks
I am a noobie in this context

I am new to rust
I am new to uv

=> I am curious about your opinion and suggestions for improvements
Not completely ready
The current version of the PR is a draft. Especially in regards of error handling and logging. Also testing can be improved.
I open this PR to discuss the direction in which the implementation is heading. As I am new to rust & uv I would also like to receive your guideance upfront 🙂

It would be nice to add a --password <password> option and store/retrieve it using the keyring crate. It should still fall back to the subprocess method for dynamic passwords.

@bschoenmaeckers, I think adding --password <password> can be seen independently of the keyring crate (https://docs.rs/keyring/latest/keyring/?).
One part is implementing an import keyring-provider. This would rely on the keyring crate. Also, it should be implemented independently of this topic, if it is required.
The other part is providing a --password option. Which could be processed with any implemented keyring-provider - currently only subprocess. I can add this part to this PR.

@stoney95 Thx for starting integration of kering-rs into uv!
How to use credentials already defined in credential store?

@zanieb, clippy is configured to not allow the usage of fs::File, fs::read_to_string & fs::remove_file (see this check). Should the async version provided by tokio be used instead? Or is there another reason why the usage is not allowed?


Syntax highlighting disappeared

I had haskell-vim working, but I no longer get syntax highlighting at all for Haskell.
I did do an update just now to your latest commit, but probably the error is on my part. Here's the .vimrc, I'm running neovim 0.2.0. https://da.gd/wkVF
I've tried disabling the theme, and most packages one by one, but I can't figure it out.

Same here, I'm also running neovim 0.2.0 and the highlighting is 'lighter' now. I'm running on today's master branch version.

I've searched a bit, it could be related to #66 in my case, TL;DR my colorscheme gruvbox overrides haskell-vim colors.

I confirmed that after getting rid of gruvbox overridings, it much better. Sorry for the noise.

Same problem here. Please have a fix for that, I’m running on a fork right now.

Thanks for reporting.

My problem was caused by the combination of intero-neovim and the line
#!/usr/bin/env runhaskell
which caused the filetype to be detected as config instead of haskell.

Is this related? Hightlighting breaks in multiline imports and ADT syntax


babel@7

What You're Solving

upgrade to use babel v7
pay down technical debt
set us up for future goals

Design Decisions

moved to using a root level babel.config.js file as per the babel recommendation for monorepos
integrated the link-parent-bin library so package directories have access to root level executables

How to Verify

npm run clean and remove all previously installed node_modules
npm install and npm run bootstrap
npm test from the root directory
npm test and npm run storybook in applicable package directories

Task Tracking
packages

[x] actionmenu
[x] avatar
[x] badge
[x] banner
[x] breadcrumb
[x] build
[x] button
[x] card
[x] carousel
[x] checkbox
[x] circularprogress
[x] core
[x] datepicker
[x] datawell
[x] dialog
[x] drawer
[x] dropdown
[x] emptystate
[x] errors
[x] filter-react-props
[x] focusmanager
[x] form
[x] halo
[x] icon
[x] layout
[x] linearprogress
[x] link
[x] normalize
[x] note
[x] packages-api
[x] position
[x] prop-types
[x] radio
[x] row
[x] searchinput
~[ ] site?~
[x] starrating
[x] storybook-addon-center
[x] storybook-addon-theme
[x] switch
[x] tab
[x] table
[x] tag
[x] text
[x] textarea
[x] textinput
[x] theme
[x] tooltip
[x] util
[x] viewtoggle

side quests/yak shaving

[x] root level babel config
[x] babel 7 and jest integration

[x] storyshots
[x] jest upgrade
[x] react-testing library upgrade

[x] storybook 5 upgrade
[ ] update .browserlistrc

[ ] add not dead to strip ie10
[ ] upgrade postcss configs

[x] update create-component template
[ ] update cra example?
[ ] glamor should be a peer dependency
[x] remove test:updateSnapshot

haha. i think that's the appropriate response. we already paired on the major changes. the rest is just cat herding


[EUWE] Add Openstack metric service to Settings

Openstack Metrics capture service can be specified in settings.yml file.
Valid values are "gnocchi", "ceilometer". All other values means autodetection.
Links

https://bugzilla.redhat.com/show_bug.cgi?id=1421729
master version of this PR https://github.com/ManageIQ/manageiq/pull/13918

@aufi Can you re-submit against master, and change it so that there's a default setting where behavior is as-is?  Thanks!

Closing in favor of #13918


Restructure and create usable sdk for other users

Now that a small version was created to send screenviews and exceptions for the other project need to restructure this sdk for others to be able to use if they want to

Currently testing prior to merging to main and creating a dll


Experiment: Show Pairing screen at ride start if initial pairing was skipped

In many cases (especially BLE), the start of a ride fails if the user previously has skipped the pairing screen.
Theory:
We could reduce the number of these startup errors, if we would force the user to pair a device before starting a ride.
Side note:
However, the user should also have the option to chose to simulate a device

implemented in 0.8.41


Group Search Not Returning

Hi,
I recently became interested in groups since I use this with grayling and they just enabled group support. However when I use a ldap browser I am not getting any returned groups. I am using Crowd 2.8.1 is it possible something changed in Crowd?

Group search should work properly:
  ldapsearch -WxD "uid=${USER_UID},ou=users,dc=crowd" -H ldap://localhost:10389 -b "dn=${GROUP_ID},ou=groups,dc=crowd"

Groups should be shown properly in an LDAP Browser, if the right DN is used.
https://github.com/dwimberger/crowd-ldap-server/wiki/DN's
Verified with Crowd 2.8.3 (latest).

If you need AD type support for groups (memberof) and nested groups, please see:
https://github.com/dwimberger/crowd-ldap-server/wiki/Configuration#user-content-configuring-memberof-ad-emulation


[Spaceship] App.get_builds limit parameter not respected

New Issue Checklist

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

Issue Description

Given an instance of Spaceship::ConnectAPI::App, if you call get_builds(limit: 1), the limit is not respected and all the builds are enumerated.
Example:
app = Spaceship::ConnectAPI::App.find("com.example")
one_build = app.get_builds(limit: 1)
# one_build.size equals the number of all available builds on App Store Connect

The code at fault appears to be here
resps = Spaceship::ConnectAPI.get_builds(filter: filter, includes: includes, limit: limit, sort: sort).all_pages

The limit parameter simply reduces the size of the page, but all_pages is called regardless of a limit being defined or not. If this is the desired behavior, it seems to defeat the purpose of having a limit parameter defined on App.get_builds at all. Using the limit parameter in this context heavily impacts performance due to the increased number of network requests required to enumerate all the builds. For example, my organization has about 255 builds for a given application and by calling app.get_builds(limit: 1) 255 individual network requests are made. I think it's reasonable to expect that giving a limit would only yield the number of results defined at this level of the Spaceship API.
This can be worked around by calling into Spaceship::ConnectAPI.get_builds directly, but I think it makes sense to enable the same behavior on App.get_builds as well.
I would be willing to do the implementation work itself, but I wanted to open up an issue first since this kind of change warrants a discussion on the direction to be taken (imo).
Command executed

N/A, this is using Spaceship directly in code
Complete output when running fastlane, including the stack trace and command used

N/A, this is using Spaceship directly in code
Environment

 ✅ fastlane environment ✅
Stack

Key
Value

OS
10.15.4

Ruby
2.6.3

Bundler?
false

Git
git version 2.24.1 (Apple Git-126)

Installation Source
/usr/local/bin/fastlane

Host
Mac OS X 10.15.4 (19E287)

Ruby Lib Dir
/System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/lib

OpenSSL Version
LibreSSL 2.8.3

Is contained
false

Is homebrew
false

Is installed via Fabric.app
false

Xcode Path
/Applications/Xcode-11.4.app/Contents/Developer/

Xcode Version
11.4

System Locale

Variable
Value

LANG
en_US.UTF-8
✅

LC_ALL

LANGUAGE

fastlane files:
No Fastfile found
No Appfile found
fastlane gems

Gem
Version
Update-Status

fastlane
2.146.1
✅ Up-To-Date

Loaded fastlane plugins:
No plugins Loaded
Loaded gems

Gem
Version

did_you_mean
1.3.0

slack-notifier
2.3.2

rouge
2.0.7

xcpretty
0.3.0

terminal-notifier
2.0.0

terminal-table
1.8.0

plist
3.5.0

multipart-post
2.0.0

word_wrap
1.0.0

public_suffix
2.0.5

colored
1.2

highline
1.7.10

commander-fastlane
4.4.6

unf
0.1.4

domain_name
0.5.20190701

http-cookie
1.0.3

faraday-cookie_jar
0.0.6

faraday_middleware
0.13.1

gh_inspector
1.1.3

multi_xml
0.6.0

rubyzip
1.3.0

security
0.1.3

xcpretty-travis-formatter
1.0.0

jwt
2.1.0

uber
0.1.0

declarative
0.0.10

declarative-option
0.1.0

representable
3.0.4

retriable
3.1.2

mini_mime
1.0.2

faraday
0.17.3

signet
0.14.0

googleauth
0.12.0

httpclient
2.8.3

google-api-client
0.36.4

emoji_regex
1.0.1

jmespath
1.4.0

aws-partitions
1.297.0

aws-eventstream
1.1.0

aws-sigv4
1.1.1

aws-sdk-core
3.94.0

aws-sdk-kms
1.30.0

aws-sdk-s3
1.61.2

nanaimo
0.2.6

colored2
3.1.2

claide
1.0.3

CFPropertyList
3.0.2

atomos
0.1.3

xcodeproj
1.16.0

unicode-display_width
1.7.0

addressable
2.7.0

tty-screen
0.7.1

tty-cursor
0.7.1

tty-spinner
0.9.3

babosa
1.0.3

excon
0.73.0

unf_ext
<IP_ADDRESS>

fastimage
2.1.7

json
2.3.0

mini_magick
4.10.1

dotenv
2.7.5

bundler
2.0.2

naturally
2.2.0

simctl
1.6.8

multi_json
1.14.1

memoist
0.16.2

os
1.1.0

digest-crc
0.5.1

google-cloud-errors
1.0.0

google-cloud-env
1.3.1

google-cloud-core
1.5.0

google-cloud-storage
1.26.0

forwardable
1.2.0

logger
1.3.0

stringio
0.0.2

ipaddr
1.2.2

openssl
2.1.2

ostruct
0.1.0

strscan
1.0.0

date
2.0.0

fileutils
1.1.0

etc
1.0.1

io-console
0.4.7

zlib
1.0.0

libxml-ruby
3.1.0

rexml
3.1.9

psych
3.1.0

mutex_m
0.1.0

ox
2.13.2

webrick
1.4.2

generated on: 2020-04-28

Thanks for the catch! I agree that limit should work similar to filter and help with trimming down the responses. I don't think @joshdholtz did this on purpose, but maybe he can chime in as he did most of the implementation!

It appears that all of the App.get_* methods behave the same way (calling .all_pages after retrieving the response).

This is still an issue; haven’t had time to dig into a good way to address this myself

@Steven0351 Will fix today! Sorry about not seeing this!

No worries @joshdholtz, I had intentions of working on it myself when I opened the issues but I haven't been able to find the time. Thanks for picking it up!


Rename management view

Fixes #3730

@Potapy4 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@ktoliver : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Verified.
@Potapy4 Thanks for submitting this update.


DetermineBonds assigning methyl carbon as tetrahedral center

The attached xyz file is for caffeine, which has no stereocenters.
>>> from rdkit import Chem
>>> from rdkit.Chem import rdmolfiles
>>> from rdkit.Chem import rdDetermineBonds
>>> mol = rdmolfiles.MolFromXYZFile('/Users/jasonb/caffeine.txt')
>>> rdDetermineBonds.DetermineBonds(mol)
>>> mol.GetAtomWithIdx(0).GetChiralTag()
rdkit.Chem.rdchem.ChiralType.CHI_TETRAHEDRAL_CCW
>>> import rdkit
>>> rdkit.__version__
'2022.09.3'

Is this expected?  Is there a good way to remove invalid chiral tags?
caffeine.txt

I notice that determineBonds calls the following
void setChirality(RWMol &mol) {
  MolOps::sanitizeMol(mol);
  MolOps::setDoubleBondNeighborDirections(mol, &mol.getConformer());
  MolOps::assignStereochemistryFrom3D(mol);
  MolOps::assignChiralTypesFrom3D(mol);
}

Is there a reason to call assignChiralTypesFrom3D after assignStereochemistryFrom3D, when the latter already calls the former internally?  I haven't looked at the RDKit stereo code in a couple of years so I don't know what parts are the 'new' stereo perception functions and which are the legacy.

@jasondbiggs yeah, I think that's likely the problem. I will fix it.


bpytop: 1.0.64 -> 1.0.65

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/bpytop/versions.
meta.description for bpytop is: "A resource monitor; python port of bashtop"
meta.homepage for bpytop is: "https://github.com/aristocratos/bpytop"
meta.changelog for bpytop is: ""
Updates performed

Version update

To inspect upstream changes

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

/nix/store/2qrgi23nmq5j67h6kj5wxwc5h61zrz1g-bpytop-1.0.65/bin/bpytop passed the binary check.

1 of 1 passed binary check by having a zero exit code.

1 of 1 passed binary check by having the new version present in output.

found 1.0.65 with grep in /nix/store/2qrgi23nmq5j67h6kj5wxwc5h61zrz1g-bpytop-1.0.65

found 1.0.65 in filename of file in /nix/store/2qrgi23nmq5j67h6kj5wxwc5h61zrz1g-bpytop-1.0.65

directory tree listing: https://gist.github.com/06444a5fa99d6e508535d51221328863

du listing: https://gist.github.com/afee2dd77ed39e5712a96769f848b6dc

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
bpytop

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/2qrgi23nmq5j67h6kj5wxwc5h61zrz1g-bpytop-1.0.65 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A bpytop https://github.com/r-ryantm/nixpkgs/archive/84261ced6c2f37d3d69edf4b77a4001b75db71af.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/2qrgi23nmq5j67h6kj5wxwc5h61zrz1g-bpytop-1.0.65
ls -la /nix/store/2qrgi23nmq5j67h6kj5wxwc5h61zrz1g-bpytop-1.0.65/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

bpytop

Maintainer pings
cc @herrwiese for testing.

Result of nixpkgs-review pr 121675 at 84261ced run on x86_64-linux 1

1 package built successfully:

bpytop

1 suggestion:

warning: missing-phase-hooks
installPhase should probably contain runHook preInstall and runHook postInstall.
Near pkgs/tools/system/bpytop/default.nix:32:3:
   |
32 |   installPhase = ''
   |   ^

Result of nixpkgs-review pr 121675 run on x86_64-linux 1

1 package built:

bpytop


OLH-1291 Onboard APAR (Provider and Assessment Register Service).

Proposed changes

Add APAR to available services.
What changed

Config updated.
Why did it change

To allow the new service to be accessed from the services list and to present a service card when the service has been visited previously.
Related links

Checklists

Environment variables or secrets

[x] No environment variables or secrets were added or changed

Testing

Currently deployed to dev or can be run locally.
How to review

We suspect the URL given for the service card is incorrect, therefore waiting for new url before proceeding.


wrong URL repository

https://github.com/kedacore/charts/blob/56d54095ea07e2a2a52c0e76eea8f692a7c5a3f6/keda/values.yaml#L7

Wrong repository URL at chart default values.yaml

This is not the wrong image URL but our new one, see #127 & kedacore/governance#16.
I've missed the README, though, which is fixed in 	https://github.com/kedacore/charts/pull/132

Oh now I see


Code Quality: Improved DependencyProperty generator

Resolved / Related Issues
The primary achievement is to reduce lines of code and verbosity when declaring DependencyProperties.
Steps used to test these changes
None

Declaration

Name
Description

Name
Name of the dependency property

PropertyChanged
Name of property changed method

IsSetterPrivate
Whether the property's setter should be hidden

IsNullable
Whether the property's setter should accepts null

DefaultValue
Default default value of the property set on initialization

Usage
[DependencyProperty<bool>("IsToggled", "OnIsToggledPropertyChanged", IsSetterPrivate = false, IsNullable = false, DefaultValue = false)]

Generated code
There's no line break inside of each class member in actual source but inserted for readability.
#nullable enable
namespace Files.App.Controls;
partial class ThemedIcon
{
    public static readonly global::System.Windows.DependencyProperty IsToggledProperty =
        global::System.Windows.DependencyProperty.Register(
            nameof(IsToggled),
            typeof(bool),
            typeof(global::Files.App.Controls.ThemedIcon),
            new global::Microsoft.UI.Xaml.PropertyMetadata(
                (bool)true,
                (d, e) => ((global::Files.App.Controls.ThemedIcon)d).OnIsToggledPropertyChanged(
                    (bool)e.OldValue,
                    (bool)e.NewValue));

    public bool IsToggled
    {
        get => (bool)GetValue(IsToggledProperty);
        set => SetValue(IsToggledProperty, value);
    }
}

Ready (tested locally) to merge.
@Poker-sang fyi. btw do you know how to emit ExpressionSyntax from TypedConstant that can be obtained from attribute properties list? This https://github.com/files-community/Files/pull/16025/commits/abb466106cec4a491169e46a3bc57869485a668f does its job but looks quite bad.

🤔 Since we've made it to the eve of the release of some of the properties, maybe we can wait for C#13 to come out before making adjustments? That way we can implement it in this syntax:
[DependencyProperty(PropertyChangedCallback, etc.)]
public partial object? DefaultValue { get; private set; } = new Object();

This way is much easier and more readable. And perhaps CommunityToolkit would like to implement a better version?

Yeah but it'll be implemented in WCT 8.x while we use 7.x and 8.x packages are incompatible with Files because some API from 7.x still missing. So I decided to make use of this for the time being (we're likely to create more controls in the near feature).
For more improvements, I'll wait until .NET 9 / C# 13 becomes GA.
I might ask you help again 😉

@yaira2 This is ready (tested locally), please rerun the CI to check.

@Poker-sang fyi. btw do you know how to emit ExpressionSyntax from TypedConstant that can be obtained from attribute properties list? This abb4661 does its job but looks quite bad.
I have
object? DefaultValue { get; set; }

But AttributeData lets me get that value as object? with some extra info of its type. do i have to use switch statement and decide the expression based on type such as enum and primitive?

If you wish to do so, then indeed you can. You can get it through ((TypeDeclarationSyntax)generatorAttributeSyntaxContext.TargetNode).AttributeLists[0].Attributes[0].ArgumentList.Arguments[0].Expression. However the arguments of the attributes only allow constants, which I don't think is very useful, so I used string

I think I'd like to use the latter one but there seems to be a cast error.

Worked thanks!!

That didn't work; somehow emitted property type string like "IconType" and caused an invalid cast exception.
Reverted to your original approach, I think we can wait for .NET partial properties to achieve this as you suggested.
Again, ready.

@0x5bfa are you ready for me to merge this?

YES!!

This PR disables changing the sort order by clicking on the header of the details layout.

Fixed


feat: ACNA-3007- update post templates action/endpoint to bypass install config

Description
When the request body to POST /templates includes apis or credentials, we need to bypass the get install config step and just save the passed request body.
The request body schema for POST /templates will need to be expanded to include the full template schema, except for id (This is set by Mongo).
This PR addresses these issues.

Related Issue
https://jira.corp.adobe.com/browse/ACNA-3007

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] I have signed the Adobe Open Source CLA.
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

UTs need to be fixed/added


Fix UWP naming of scaled assets

File names of scaled assets must include a dot (.) in front of the scale-xxx suffix.

Thanks again!


Update to github.com/mitchellh/hashstructure v2

This repository has been archived by the owner on Jul 22, 2024. It is now read-only.

https://github.com/mitchellh/hashstructure


Frontend

Hello @jhadev
Will there be any content inside your empty issue description?

I'm sorry this was mistake on the mobile app.


fix: blank iframe issue

Description
fixes #15
iframe onLoad isn't working
What

set inner html in useeffect hook

Can you please add a before and after? I'm curious why setTimeout of 200ms. Why not directly use the useEffect hooks?

Before

After

As for setTimeout, I have made mistake.

@moogii Thanks for the PR.
Can you please test it with other browsers, I've used this approach before. It wasn't working.

I read all comments on #15. Maily isn't working on Chrome (Version 118.0.5993.117 (Official Build) (arm64)) and Safari (Version 17.0 (196<IP_ADDRESS>.1)). Even after disabling extensions iframe was blank also I haven't installed any extensions on Safari. However it is working without issue on Brave though. After digging around I found that I frame is not consistent through browsers and OSs.

It's not working in Firefox.

Interesting. Maily preview is working properly in Firefox.
Now my solution is working in Firefox.

Check my changes, it will work now work on Safari and Firefox properly.


$. could default to $.0

E.g. using
.+
$.

instead of
.+
$.0

I'll think about it. I don't want to go too overboard with these shorthands, but this one seems useful.

For your specific example it's shorter to do M`. though, or M%`. if there can be multiple lines in the input.


Conversion to pdf is okay, but conversion to default filetype like .txt is giving empty file.

tesseract scan.tif filename This command is giving empty text file.
But this tesseract scan.tif filename pdf is giving correct pdf file. What can be the issue ?

try tesseract scan.tif filename -psm 1

Please use tesseract forum for asking support: https://github.com/tesseract-ocr/tesseract/wiki/FAQ#rules-and-advice


fix(Geosuggest): fix accessibility

Description
This PR fixes an accessibility with the current solution

listbox field does not have a proper label

Checklist

[x] All tests passing
[x] Created tests which fail without the change (if possible)
[x] Extended the README / documentation, if necessary (N/A)
[x] Commits and PR follow conventions

Fixes :

https://github.com/ubilabs/react-geosuggest/issues/478

Hello @yfr any chance we could give the accessibility PR's a quick look ? ;) Would be great thank you!

Hey @plumdumpling! Thanks a lot for finally merge this fix!
Any chance you could release the accessibility fix with a patch version?

Hey @fcabreraupgrade! We would like to release the fix together with some more fixes and smaller updates. We're planning to have a new release ready this week.

@plumdumpling thanks a lot!


German writing conventions uses a comma (+fix included)

Hello,
German writing norms use a comma instead of a dot to separate numbers such as "1.1/1,1". This is a regular issue on german realms, as "1,6k" will register as "6k".
I made a fork and a simple change in the RegEx to GDKPd.lua to fix this, you can find it here: https://github.com/NoTaubsi/GDKPd-Classic/commit/fa076962d0f2ffd9cb788502905012b77b4695c5
This keeps the current functionability, but adds the feature of using commas.
I'm new to GitHub, I hope you find this helpful. Your Discord invite link is out of date by the way.

I wasn't able to get this code to work for me locally (enUS locale). I recognize the need for this functionality though and it's even an issue on the US realms. I'll look into it more closely soon.


New version: Ryujinx.Ryujinx version 1.1.1298

Created by 🥟 Dumplings in workflow run #1271.
Logs
Updated: 1.1.1297 → 1.1.1298
Cannot validate argument on parameter 'ChatID'. The argument is null, empty, or consists of only white-space characters. Provide an argument that contains non white-space characters, and then try the command again.
Submitting WinGet manifests
Cannot validate argument on parameter 'ChatID'. The argument is null, empty, or consists of only white-space characters. Provide an argument that contains non white-space characters, and then try the command again.

Microsoft Reviewers: Open in CodeFlow

Automatic Validation ended with:

Executable C:\Users\validator\AppData\Local\Microsoft\WinGet\Links\Ryujinx.exe returned exit code: -532462766

(Automated response - build 880.)


Icon Request: Mortar and Pestle (compounding)

Request: A mortar and pestle to symbolize drug compounding.
Use case: I am creating a page with pediatric drug dosing guidelines and am in need of icons to symbolize the different formulations of icons. I have tried to make do with emoji but there is an issue. The categories are:
Tablets/capsules (💊), suspensions (🧪), and extemporaneously prepared (i.e., compounded) solutions (no suitable emoji available)
These different formulations have very different implications for patient suitability, access and insurance coverage
Example: Here is relatively a clean example of an outline mortar and pestle vector image.

Thanks for all your work. Big fan of the icon set.

This is now done and will go out in the next release in a week or two.

Attaching the SVGs in case you need them sooner. Thanks for your patience.

Thank you so much! These are great. I really appreciate your work.
On Mon, Jan 8, 2024, 09:55 Daniel Burka @.***> wrote:

Attaching the SVGs in case you need them sooner. Thanks for your patience.
medicine_mortar_filled.svg (view on web)
https://github.com/resolvetosavelives/healthicons/assets/711809/aff4fca9-e278-4943-8589-db62cfb1e283
medicine_mortar_negative.svg (view on web)
https://github.com/resolvetosavelives/healthicons/assets/711809/5ec1520e-7767-4ec4-8d65-41d4337023d1
medicine_mortar_outline.svg (view on web)
https://github.com/resolvetosavelives/healthicons/assets/711809/cadc7f30-41ca-4114-8177-3c186e66f6d9
—
Reply to this email directly, view it on GitHub
https://github.com/resolvetosavelives/healthicons/issues/179#issuecomment-1881468458,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXMW77IYI7MAIS3NNRIWTWDYNQQIRAVCNFSM6AAAAAAXGDYZISVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOBRGQ3DQNBVHA
.
You are receiving this because you authored the thread.Message ID:
@.***>


fixed bug (corrected the path to profile when clicking account)

Summary by CodeRabbit

Bug Fixes

Corrected a typo in the routing declaration to ensure proper navigation within the app.

New Features

Updated navigation links: The "account" link in the dashboard now correctly points to the user's profile page.

Should not have changes in package file, also check the typo

@youssefjaziri456, any updates?

@pranshugupta54 Yes I corrected the typo and changed the path to profile with the link that you gave me ,will be making a pull request soon.


run error: cannot import react-native-root-toast

I fork odota/mobile and run for the first time today.
An error related to import Toast from 'react-native-root-toast' appeared, as shown in attached image

The workaround for this bug is similar to this comment
https://github.com/magicismight/react-native-root-toast/issues/88#issuecomment-428424540

This bug occurs when I update package using npm
With yarn, it is OK.
I built on Mac, IDE is WebStorm.
Anyone knows why my pull request is failed when build with travis-ci?

This bug occurs when I update package using npm
With yarn, it is OK.
I built on Mac, IDE is WebStorm.
Anyone knows why my pull request is failed when build with travis-ci?


[Bug] Got "Already assigned" exception after sending the empty string to the chat

🐛 Bug
Got exception after inputting the empty string to the chat
To Reproduce
Steps to reproduce the behavior:

Mac
Follow the instruction:

# Create a new conda environment and activate the environment.
conda create -n mlc-chat
conda activate mlc-chat

# Install Git and Git-LFS if you haven't already.
# They are used for downloading the model weights from HuggingFace.
conda install git git-lfs
git lfs install

# Install the chat CLI app from Conda.
conda install -c mlc-ai -c conda-forge mlc-chat-nightly --force-reinstall

# Create a directory, download the model weights from HuggingFace, and download the binary libraries
# from GitHub. Select one of the following `LOCAL_ID` for a prebuilt LLM.

mkdir -p dist/prebuilt
git clone https://github.com/mlc-ai/binary-mlc-llm-libs.git dist/prebuilt/lib

# Download prebuilt weights of Vicuna-7B
cd dist/prebuilt
git clone https://huggingface.co/mlc-ai/mlc-chat-vicuna-v1-7b-q3f16_0
cd ../..
mlc_chat_cli --local-id vicuna-v1-7b-q3f16_0l

Expected behavior
A friendly output.
Environment

Platform:
Operating system Mac
Device: Mac M1
How you installed MLC-LLM: conda

Additional context
Stack Trace:
USER: could you help me
ASSISTANT: Of course! I'd be happy to help you with any questions you have. What can I help you with?
USER:
[23:07:36] /Users/catalyst/Workspace/miniforge3/envs/mlc-llm-build/conda-bld/mlc-chat-nightly-package_1685952072752/work/cpp/conversation.h:161:
---------------------------------------------------------------
An error occurred during the execution of TVM.
For more information, please see: https://tvm.apache.org/docs/errors.html
---------------------------------------------------------------

  Check failed: this->messages.back().size() == 1 (2 vs. 1) : Already assigned
Stack trace:
  [bt] (0) 1   libmlc_llm.dylib                    0x00000001051e65f0 tvm::runtime::detail::LogFatal::Entry::Finalize() + 68
  [bt] (1) 2   libmlc_llm.dylib                    0x00000001051e65ac tvm::runtime::detail::LogFatal::Entry::Finalize() + 0
  [bt] (2) 3   libmlc_llm.dylib                    0x00000001051e57c8 __clang_call_terminate + 0
  [bt] (3) 4   libmlc_llm.dylib                    0x0000000105206c60 mlc::llm::Conversation::FinishReply(std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char>>) + 440
  [bt] (4) 5   libmlc_llm.dylib                    0x0000000105203480 mlc::llm::LLMChat::ProcessNextToken(int) + 1660
  [bt] (5) 6   libmlc_llm.dylib                    0x0000000105201be8 mlc::llm::LLMChat::PrefillStep(std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char>>) + 708
  [bt] (6) 7   libmlc_llm.dylib                    0x00000001052017a4 mlc::llm::LLMChatModule::GetFunction(tvm::runtime::String const&, tvm::runtime::ObjectPtr<tvm::runtime::Object> const&)::'lambda2'(tvm::runtime::TVMArgs, tvm::runtime::TVMRetValue*)::operator()(tvm::runtime::TVMArgs, tvm::runtime::TVMRetValue*) const + 140
  [bt] (7) 8   libmlc_llm.dylib                    0x000000010520170c tvm::runtime::PackedFuncObj::Extractor<tvm::runtime::PackedFuncSubObj<mlc::llm::LLMChatModule::GetFunction(tvm::runtime::String const&, tvm::runtime::ObjectPtr<tvm::runtime::Object> const&)::'lambda2'(tvm::runtime::TVMArgs, tvm::runtime::TVMRetValue*)>>::Call(tvm::runtime::PackedFuncObj const*, tvm::runtime::TVMArgs, tvm::runtime::TVMRetValue*) + 40
  [bt] (8) 9   mlc_chat_cli                        0x00000001048490e4 Converse(ChatModule*, std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char>> const&, int, std::__1::basic_ostream<char, std::__1::char_traits<char>>&) + 136

Probably it's an edge case we want to patch @yzh119

Should fixed by #328


[20.09] go-ethereum: 1.9.20 -> 1.9.24

Motivation for this change
Backports #103653
1.9.24 contains a consensus fix for a problem that will arise on or around 1st January 2021
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 103900 1

1 package built:
- go-ethereum


bug: manifest has different id than the settings import

https://github.com/AceLikesGhosts/ytm-rpc/blob/master/client-mods/replugged/manifest.json#L2
should be all lowercase

will fix whenever i do any changes to the codebase

resolved


Also check if saver is a permitted user for m.NOTICE

Previously we would only have checked for m.MESSAGE. As an aside, we only check for is_text() events now, so technically this allows saving an emote as well. There is no reason why we should exclude those...
I have to admit I don't understand what the bot check in m.NOTICE checks is supposed to do or what it protects against, so I just kept it in.
Fixes #16

thanks for this!
the check on NOTICE type messages prevents the bot from saving a response that came from another bot. most bots send their text messages as type m.notice, and because they are messages that don't actually need to be saved because they are automatically generated, i prevent this.
my community likes to save everything and do unofficial QA on my bots :laughing: , so effectively this prevents a circular dependency of having the bot save a quote of a quote that the bot has just returned.

hmm, i should have tested this before merging... getting the following error in my maubot which uses tulir's official docker container:
Traceback (most recent call last):
  File "/usr/lib/python3.10/site-packages/mautrix/client/syncer.py", line 235, in _catch_errors
    await handler(data)
  File "/opt/maubot/maubot/handlers/command.py", line 480, in replacement
    await func(self, evt, val)
  File "/data/plugins/org.jobmachine.gifme-v0.0.16-ts1677621740567.mbp/gifme/bot.py", line 340, in save_react
    await self.save_msg(source_evt)
  File "/data/plugins/org.jobmachine.gifme-v0.0.16-ts1677621740567.mbp/gifme/bot.py", line 174, in save_msg
    elif source_evt.content.msgtype.is_text():
TypeError: 'bool' object is not callable

i've reverted this because it broke my bot. will need to test further.