Add SQL Example that using a query as well as a table connection

[Enter feedback here]
Can an option for the SQL code, example be added to show the option that a query can be used. The Python and Spark example contain an example using a direct table connection and a query to get the data. The same should be done for the sql one, as its not quite clear that you can do it that way
Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 358fe757-e92a-30d5-300b-2911e18dfd4c
Version Independent ID: 358fe757-e92a-30d5-300b-2911e18dfd4c
Content: Query data in Azure Synapse Analytics - Azure Databricks
Content Source: databricks/external-data/synapse-analytics.md
Service: azure-databricks
GitHub Login: @mssaperla
Microsoft Alias: saperla

@mrjonlunn
Thanks for your feedback! We will investigate and update as appropriate.

@mrjonlunn
Thank you for bringing this to our attention.
I've delegated this to content author @mssaperla, who will review it and offer their insightful opinions.

@mssaperla
Could you please review add comments on this, update as appropriate.

Thanks for providing feedback that helps improve our documentation. We've created an internal work item (DOC-8887) to address your feedback. The timeline for resolution varies based on resourcing.
#please-close


Update action name from "New version" to "Edit model"

Current:
To create a new version, select New version at the top of the page.
Edit:
To create a new version, select Edit model at the top of the page.

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

ID: 1dc86433-875e-b3c4-0d4d-f21e14a27158
Version Independent ID: b55397ee-0c6b-b3b4-81ee-6573752b5bc3
Content: Manage your model in AI Builder - AI Builder
Content Source: ai-builder-docs/manage-model.md
Service: aibuilder
GitHub Login: @paulnog
Microsoft Alias: paulnog

@cimaja - Thanks for your feedback. We have updated the docs to fix https://docs.microsoft.com/en-us/ai-builder/manage-model#create-a-new-version, and the updated content will be live today. Closing this issue.


fix(bvp): don't use numeric limits for start/finish distance

Description

Remove redundant process calling updateRTCStatus.
Don't use std::numeric_limits<double>::lowest() for start/finish distance in order to avoid ros2 command crashing. https://github.com/ros2/rclpy/issues/848

Tests performed

[x] PASS TIER IV INTERNAL SCENARIOS

Effects on system behavior

Nothing.
Interface changes

Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

@rej55 Thanks. I think there is no problem since updateRTCStatus is always called in sendRTC function per every planning cycle.
On the other hand, I found it was able to echo topics that include nan or inf. Then, I close this PR.


4268 edit media info entity

Fixes #4268
Changes proposed in this pull request:

Make WbItemDocumentExpr extend EntityIdValue instead of ItemIdValue.
Modify corresponding tests.

@j-sal amazing! It looks like this pull request also includes your previous changes, which have already been merged into master. It is worth making sure you update your master branch locally (git checkout master ; git pull) before you start working on a new pull request. For now we can solve this simply by pushing the "Update branch" button which will simply merge the master branch in your pull request branch.

Another small point: it is helpful if you give an easy to read title to your pull request. By default, GitHub draws it from your branch name (if you are pushing multiple commits), so it currently is "4268 edit media info entity": you could make it more descriptive such as "Make WbItemDocumentExpr extend EntityIdValue instead of ItemIdValue"


The “most recent” filter is not working correctly.

As you can see the nodes are not sorting correctly
https://user-images.githubusercontent.com/1402054/172209423-8ec8d7f9-27b0-4512-ac31-bbc549dd4de1.mov

Currently it is using the field updatedAt.
Use the field changedAt.
Dont use updatedAt anywhere in the code base


model load error

I train senet model like this:
senet =SEResNeXt.SEResNeXt(224,12).model
adam = tensorflow.keras.optimizers.Adam(lr=0.001, beta_1=0.9, beta_2=0.999, decay=1e-6)
 senet.compile(loss='categorical_crossentropy',optimizer=adam, metrics=['accuracy'])
 filepath="model_val/weights-improvement-{epoch:02d}-{acc:.2f}.hdf5"
 checkpoint = ModelCheckpoint(filepath, monitor='acc', verbose=1, save_best_only=True, mode='max')
 senet.fit(X, Y ,epochs=200, batch_size=2,callbacks=[checkpoint ])
When I try to load trained model, i got this :
    model = load_model('model_val/weights-improvement-32-0.95.hdf5')   File "/home/tensorflow04/anaconda3/lib/python3.6/site-packages/keras/models.py", line 240, in load_model     model = model_from_config(model_config, custom_objects=custom_objects)   File "/home/tensorflow04/anaconda3/lib/python3.6/site-packages/keras/models.py", line 314, in model_from_config     return layer_module.deserialize(config, custom_objects=custom_objects)   File "/home/tensorflow04/anaconda3/lib/python3.6/site-packages/keras/layers/__init__.py", line 55, in deserialize     printable_module_name='layer')   File "/home/tensorflow04/anaconda3/lib/python3.6/site-packages/keras/utils/generic_utils.py", line 139, in deserialize_keras_object     list(custom_objects.items())))   File "/home/tensorflow04/anaconda3/lib/python3.6/site-packages/keras/engine/topology.py", line 2490, in from_config     process_layer(layer_data)   File "/home/tensorflow04/anaconda3/lib/python3.6/site-packages/keras/engine/topology.py", line 2476, in process_layer     custom_objects=custom_objects)   File "/home/tensorflow04/anaconda3/lib/python3.6/site-packages/keras/layers/__init__.py", line 55, in deserialize     printable_module_name='layer')   File "/home/tensorflow04/anaconda3/lib/python3.6/site-packages/keras/utils/generic_utils.py", line 139, in deserialize_keras_object     list(custom_objects.items())))   File "/home/tensorflow04/anaconda3/lib/python3.6/site-packages/keras/layers/core.py", line 696, in from_config     printable_module_name='function in Lambda layer')   File "/home/tensorflow04/anaconda3/lib/python3.6/site-packages/keras/utils/generic_utils.py", line 156, in deserialize_keras_object     fn = module_objects.get(function_name) AttributeError: 'NoneType' object has no attribute 'get'

@gftd
It seems your environment is different from mine, I cannot reproduce your error.
And I just checked a trained model loading in https://github.com/yoheikikuta/senet-keras/blob/master/evaluate-cifar10.ipynb; in the notebook I loaded the model as follows:
model = SEResNeXt(size=size).model
model.load_weights("trained_model/SEResNeXt.h5", by_name=True)

I'm not sure whether it is helpful to you, but maybe model instantiation -> load_weight step is required.

could you give us the link to download the pretrained senet model of keras version?

Since I provided a trained model, I'll close this issue.


Flow says function is missing, when actually it's just called incorrectly

Hello!
It seems like there is a bug/change in behavior when a function is called with invalid arguments. Currently flow error says that the function cannot be found in the props object when in fact it IS there, just called incorrectly.
TryFlow
In the example above we get a different error than what we would get in v0.75
// v0.101.0
all branches are incompatible: Either property `submit` is missing in `OwnPropsT` [1]. Or property `submit` is missing in `ReduxPropsT` [2].

// v0.75.0
Cannot call `submit` with object literal bound to `data` because property `code` is missing in object literal [1] but exists in object type [2].

TryFlow
In other example, there is no error in the latest version but we get an (correct) error in v0.75
// v0.101.0
No errors!

// v0.75.0
Cannot call `foo` with object literal bound to `data` because property `code` is missing in object literal [1] but exists in object type [2].

Thanks!

In your second example, this is the expected behavior because of how Flow treats unsealed objects. See #7345.
The first example appears to be a poorly positioned error due to destructuring, which I believe we are working on fixing.

So for the second example, if I make the parameter object an exact type but the property inside optional, should I be getting an error?
TryFlow

Yes, because {} is an inexact object.

Hmm even if code is "optional"? Or is flow doing away with optional object properties?
TryFlow

Exactness and inexactness are separate concepts from optional properties. An inexact object can never be assigned to an exact one, even if they have the same properties: https://flow.org/try/#0DYUwLgBAHhBcEG8CGcIGYC+EC8iX0wG4AoY0SAT1QQB990atdlUjiIPPPircpCugoA

I understand the exact and inexact concepts. My main question is how do optional properties fit in with exact types? Or do they not?
https://github.com/facebook/flow/issues/2386

Fixed in 0.191.0


Add models and examples reference

Figured I'd tackle this before we eventually get to the release date and we still haven't done it.
Checklist

[x] Made sure the book is up to date with changes in this PR.

Related Issues/PRs
Closes #1808
Closes #1809
Changes

Added reference to the models repo in the book
Added the list of examples in the book and README

When rendered, the examples section looks like this:

Excellent, I made optional suggestions.

Hah, didn't know we could directly suggest and then commit the suggested changes 🤯
Suggestions applied! The examples section was your idea after all, so thanks for the feedback 🙏


GraphQL mutation to for user to complete survey

[ ] When a user completes a survey, we will want to have a new mutation that can change the survey data inside of the users table from true to false.

Mutation has been added, and tested with FE object to save to our DB


Could you please share the FPS benchmarking results of your experiments?

Hi, thanks for your question. I will update soon with timing results.


Unknown class expression in oboInOwl property

https://www.ebi.ac.uk/ols4/ontologies/oio/properties/http%253A%252F%252Fwww.geneontology.org%252Fformats%252FoboInOwl%2523hasSynonymType?lang=en

See value of has_dbxref on the right

might be related to https://github.com/EBISPOT/ols4/issues/717


Use template when generating tempdir in verify-crds

Pull Request Motivation
Due to a bug in controller-gen certain paths are incorrectly split and part of these paths can be interpreted as a numeric literal, which will cause controller-gen to fail. We observe this as occasional test flakes in the "verify-crds" target, when the tmpdir starts with a zero, such as in /tmp/tmp.0PFqFSHBID
This commit attempts to avoid this bug by specifying a template for the tmpdir we generate when verifying CRDs which doesn't include any "." characters, which seem to be being split incorrectly.
Kind
/kind bug
Release Note
Use manually specified tmpdir template when verifying CRDs

/cherry-pick release-1.10

/cherry-pick release-1.9

@SgtCoDFish can you confirm that this template prevents all possible errors?
eg. does "tmp-CHECKCRD-0XXXXXXXX" not cause problems?
/lgtm
/approve
/hold

@SgtCoDFish can you confirm that this template prevents all possible errors? eg. does "tmp-CHECKCRD-0XXXXXXXX" not cause problems?

Can confirm. If you change the line to tmpdir="$(mktemp -d tmp-CHECKCRD-0XXXXXXXXX --tmpdir)" it'll work fine.
If you change it to tmpdir="$(mktemp -d tmp-CHECKCRD.0XXXXXXXXX --tmpdir)" it works fine.
If you change to tmpdir="$(mktemp -d tmp.CHECKCRD-0XXXXXXXXX --tmpdir)" it works fine.
If you change to tmpdir="$(mktemp -d tmp.CHECKCRD.0XXXXXXXXX --tmpdir)" it fails reliably.
/unhold


Firebase authentication and database access

I used the prescribed method of authentication on Firebase as below (I'm using version 1.x as I'm on PHP v 5.6 and I understand the latest firebase-php version requires PHP v 7)
use Kreait\Firebase\Configuration;
use Kreait\Firebase\Firebase;

$config = new Configuration();
$config->setAuthConfigFile(__DIR__.'/google-service-account.json');

$firebase = new Firebase('https://<my-app>.firebaseio.com', $config);
$firebase->get('/');

At first I had a permission denied error, but after several frustrating attempts to overcome it, I decided to make my read access public, but even after doing so, I was returned with an empty json. As I understand now, I have 2 issues - the first being an authentication issue, and later on a data retrieval issue.
I have tried successfully using NodeJS and the same credentials to access the same firebase database and am quite perplexed with regard to the current issues. The difference (as I noticed) is that in NodeJS, I first initialized my firebase app with the configuration json, then authenticate myself by signing in with email and password before I proceed to retrieve data.
Is there a possibility that version 1.x might have some peculiarities that need some special configuration?

That's really strange, I have no clue why the same Credentials File would work in one project but not in the other, so I can only guess: Perhaps there is a protection on Firebase's side that prevents the usage of a credentials file in two different environments or at the same time.
In general I think it would be good advice to use separate credentials for separate projects, so if you haven't done so already, I suggest you create new credentials and try with them.
At this point, I don't think it's an issue with the library itself, as there haven't been changes since some time and the problem does not to be widespread among the users of the SDK.
Please let me know if you found out the reason, this is an intriguing issue.

Hi Guys, I've been having the same problem. Some one figured out the reason?

If you can connect to the Firebase Database at all, it means that the service account JSON itself is valid and working.
The fact that you need to authenticate as an user (in the Node environment by entering an email and a password) is a hint that the Service Account doesn't have enough permissions by itself.
Please make sure in the Google IAM that the service account has the "Project -> Owner" Role.


BSP missing the VL53L0X ToF distance sensor files

Caution
The Issues are strictly limited for the reporting of problem encountered with the software provided in this project.
For any other problem related to the STM32 product, the performance, the hardware characteristics and boards, the tools the environment in general, please post your report to the ST Community in the STM32 MCUs dedicated page.
Describe the set-up

The board (B-L4S5I-IOT01A)
IDE 1.11.2

Describe the bug (skip if none)
The BSP should include all sensors on the board and VL53L0X sensor is on the board but doesn't have a drivers files inside of BSP
How to reproduce the bug (skip if none)

Indicate the global behavior of your application project
List the modules that you suspect to be the cause of the problem (Drivers, BSP, MW...)
Describe the use case that generates the problem
How we can reproduce the problem

Additional context
please developed and update the BSP files to include and integrate the VL53L0X sensor into the BSP files like the ones from here
https://github.com/wmdscjhdpy/VL53L1X_STM32_module
https://www.st.com/en/imaging-and-photonics-solutions/vl53l0x.html?rt=db&id=DB4184
Screenshots
If applicable, add screenshots to help explain your problem.

BSP files missing BL53L0X

Hello @JustinAdmin,
Thank you for your report. We do not have a specific example for the VL53L0X sensor on the B-L4S5I-IOT01A board, but you can find the driver for the VL53L0X from an example developed for the B-L475E-IOT01A here.
You may need to make the necessary modifications to adapt it to your project.
Please let us know if you require further assistance.
With regards,

Hello @JustinAdmin,
Any update regarding this thread?
With regards,

Hello @JustinAdmin,
Please allow me to close this thread as no activity. You may reopen it at any time if you have any details to share with us. Thank you for your comprehension.
With regards,


Input Fields to Update Drawing

When users update the input fields for the rows they must press the ENTER key to submit each update for the drawing.  See about updating onBlur when the field loses focus.

Input fields function on blur, keypress and change.  This executes an update to the drawing as expected.  The issue of updating the angle labels for the input fields remains.


Fixed broken URL to "Deep Learning for Web Search and Natural Language Processing" slides.

Fixed broken URL to "Deep Learning for Web Search and Natural Language Processing" slides.

thank you :)


Module name for import seems to be 'bitcoin' not 'pybitcointools'

Running:
2.7.6 |Anaconda 1.9.2 (x86_64)| (default, Jan 10 2014, 11:23:15)
[GCC 4.0.1 (Apple Inc. build 5493)]

from pybitcointools import *
ImportError: No module named pybitcointools
from bitcoin import *
(success)

...might want to edit README.
Thanks VB, such great work

Maybe this could cause problems with https://github.com/petertodd/python-bitcoinlib , which though  uses a different name on pypi, installs locally  to  bitcoin :(


feat!: upgrade to AWS CDK v2

Update Construct Hub to use AWS CDK v2.

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

closes #108


split path in unix and windows

as suggested by @tst2005 here: https://github.com/Tieske/Penlight/pull/331#issuecomment-667939801
instead of the path functions operating on the current system (either Windows or Unix), create 2 namespaces; path.unix and path.windows that implement their respective versions, and set the system one as the default.
Questions;

is this breaking?
does thsi require the application to understand the platform difference? idea was probably that it would be transparent to the aplication...

As long as the original function name was linked to the platform specific function and Penlight was doing the platform detection I don't see why this would be breaking or require any understanding on the part of the application.

@Tieske if you like one of my idea ... maybe you will be interested by the others ...
2 years ago, I made some experimentation on a forked Penlight named litepain.
I was pretty sure my approach will only usefull for me, but maybe I'm wrong.

What is cool in Penlight ?
There are plenty of usefull code !

If you only need one small function/feature you need to get the whole projet.
I see some projet copy/paste the function of feature inside he projet.
I think this way to do is bad, because there is no channel for futur update of this part of code!

My experimentation was :

analyse the dependencies

make a try with pl.path to split each function(or feature) in one "standalone" sub-module file.
See the result : https://github.com/tst2005/lua-litepain/tree/litepain/lua/lp/path
Each sub-module contain the load of dependencies, his own code, return the function.

Why do that ?!

Because I search a way to be able to take part of penlight, build an all-in-one special module file and include this stuff into my own projet.
I already made lot of experiments about that...

It will be easier for me if the Penlight's code will be more "pickable".
I know it is maybe a hard work... and most of the Penlight users will don't see the différence.
I just spoke about the concept.
Conclusion of my linepain experimentation :

the main difficulty was not technical, it was to find a good way to generate documentation (where to put the ldoc block).
there is some f*cking hard case where module a cross loaded, cross-dependencies ... I try some experiementation, but it seems not the best clean way...

I'm still open for discuss.
Enjoy!

This issue is relevant I think: https://github.com/lunarmodules/LDoc/issues/387
normalization depends on the use case;

do 2 paths point to the same file?
should it be normalized against existing paths? (currently on windows it becomes lowercase, should it normalize to the actual existing path segments as they exist on disk?)
target file systems might be case (in)sensitive; so different from the running platform


Reintroduce an improved contributing page in the documentation

Description
This PR reintroduces an improved version of the contributing page in the new documentation.

/rebase test=true


Seeking NodeJS + TypeScript Demo Project with CJS and ESM Exports

Hi,
Does anyone have a demo NodeJS + TypeScript project that serves as a best practice example for exporting with both CJS and ESM formats?
I've been unable to find a clear way to build a project that supports both pure CJS and pure ESM exports.
I believe there should be three types of project templates:

CJS masquerading as ESM
ESM masquerading as CJS
Both pure ESM and pure CJS

Is it possible for a clean project to maintain a single set of typings? It seems like ESM and CJS might each require their own set of typings.
Thank you!

ESM and CJS do necessarily require their own set of typings. I suggest looking at https://github.com/isaacs/tshy as a tool to generate both formats. However, I also suggest considering shipping ESM only as a best practice.

What @andrewbranch said. There's also https://tsup.egoist.dev/ for bundling ESM and CJS formats. You can do this pretty easily with their config:
import { defineConfig } from "tsup";

export default defineConfig({
  entry: ["src/index.ts"],
  splitting: false,
  format: ["esm", "cjs"],
  sourcemap: true,
  clean: true,
  dts: true,
});

If you don't want yet another tool then use this as a guide:

File Ext
Package Type

CJS
.js, .d.ts
CommonJS

ESM
.mjs, .d.mts
CommonJS

CJS
.js, .d.ts
module

ESM
.cjs, .d.cts
module


Change default division page

With the new navigation, the default division page is mostly redundant, and we could make a different page such as standings the default. The 'useful information' could be in a separate page, folded into another page, or embedded into each page.

If anything I think I would want to add a new "Division Hub" button to the new navigation. I have some plans that I would like to work on to make the Division Hub unique (Hall of Fame, Banners, etc) and worthy of being the default page

can we change the default page for now, until the new 'division hub' page exists?


Butcher_tableau and SImulation_data files

Hello Mr. Zhao
What are the Butcher_tableau and Simulation_data consist of? How can they derive?

Hello Dr. Zhao
According to your paper, the simulation_data file consists of peak values of inductor current and output voltage of transient state due to arbitrary load changes, but how did you change the loads? How can the peak-to-peak values obtain from simulation files?


Allow specifying the "root" name

For the cell-densities cell-density and cell-densities  inhibitory_and_excitatory_neuron_densities commands.  It defaults to "root", like for the AIBS hierarchy, if not specified.

Codecov Report
All modified lines are covered by tests :white_check_mark:

:exclamation: No coverage uploaded for pull request base (main@b1a58cc). Click here to learn what that means.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #50   +/-   ##
=======================================
  Coverage        ?   97.77%
=======================================
  Files           ?       22
  Lines           ?     1391
  Branches        ?        0
=======================================
  Hits            ?     1360
  Misses          ?       31
  Partials        ?        0

Flag
Coverage Δ

pytest
97.77% <0.00%> (?)

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

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

I reran cell-density, glia-cell-density and inhibitory-neuron-densities using the pre-commit code, and this code, with the stock AIBS hierarchy.  The densities matched for:
    'cell_density.nrrd',

    'astrocyte_density.nrrd',
    'glia_density.nrrd',
    'microglia_density.nrrd',
    'neuron_density.nrrd',
    'oligodendrocyte_density.nrrd',

    'gad67+_density.nrrd',
    'pv+_density.nrrd',
    'sst+_density.nrrd',
    'vip+_density.nrrd',

I then changed the hierarchy root name to "asdf" (and the first_estimates.csv), and reran it; and the above outputted densities were the same.


Graph viz!

Potentially add this resource for visualizing graphs https://python-graph-gallery.com/

Hi @clarecorthell,
Thanks for your interest for this gallery! Just to say there is an equivalent in R as well: https://www.r-graph-gallery.com. (Disclaimer: I'm the creator of both).
Thanks for this repo!

@holtzy this repo has now been significantly restructured! we still recommend one or two best resource(s) from the open source options available. do you continue to suggest these for visualization tutorials, referencing viz types, or something else? thanks for your patience.

Unsubscribe me
On Tue 18 Apr 2023 at 01:54, Clare @.***> wrote:

@holtzy https://github.com/holtzy this repo has now been significantly
restructured! we still recommend one or two best resource(s) from the open
source options available. do you continue to suggest these for
visualization tutorials, referencing viz types, or something else? thanks
for your patience.
—
Reply to this email directly, view it on GitHub
https://github.com/datasciencemasters/go/issues/136#issuecomment-1512287391,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFNKI4H3B5ASOXXWPR4MDJDXBXQ4TANCNFSM4EVU2QHA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Hi @clarecorthell,
Yes, I still actively maintain those resources and recommend them! On top of the R and Python graph galleries, I also recommend data-to-viz.com and dataviz-inspiration.com


Selection improvements for the capture window.

Enable highlighting selected function calls in both the capture window and the live functions data view panel.

When picking an interested function in the live function data view panel, all calls of the same function will be highlighted in the capture window.
When picking an interested function call in the capture window, the corresponding function in the live function data view panel will be highlighted.

Fixes b/168104538.

Hi Tingwei, thanks for the changes, trying out the change and I noticed a bug:

becomes:

The manually instrumented scopes are all highlighted. You can reproduce by profiling OrbitTest for example, which is compiled by default and is present next to the OrbitService executable.

Hi Tingwei, thanks for the changes, trying out the change and I noticed a bug:

becomes:

The manually instrumented scopes are all highlighted. You can reproduce by profiling OrbitTest for example, which is compiled by default and is present next to the OrbitService executable.

Hi Pierric,
Thanks for helping me test the change. I have profiled OrbitTest. This is not a bug. I have output the function addresses of the highlighted function and the clicked text box. They are both correct.
When picking the 1st function in the live Tab or picking the corresponding text boxes in the capture window,  not all manually instrumented functions are highlighted. Only the functions whose addresses are the same with the function picked in the capture window or the live Tab are highlighted.

When picking the 2nd function in the live Tab or picking the corresponding text boxes in the capture window, still only functions with matching function addresses will be highlighted.

It is true that under certain condition, many manually instrumented functions are highlighted. But this case is normal, because all the highlighted functions have the same address. (see the following 3 figures)

Here is one more example to show many functions are highlighted, but all their function addresses match our selected function addresses.

Thanks again for helping me test the code change.

Hi Tingwei,
Thanks for looking into it. I think highlighting all manually instrumented functions could be misleading.
I looked into it a bit more on my side, please see suggestions in https://github.com/pierricgimmig/orbit/commit/6cc69da06e5142d4c1e275a4a63067d815b66a77.
Feel free to incorporate in your PR.


Enum class Button::Type

Convert Button::Type from enum to enum class.
There are many other similar enums, which will eventually be converted. I'm starting with this one as a template. The bulk of the changes are simply adding the enum name prefix when accessing enum values. That can be done safely before switching to an enum class. Any code that relies on implicit conversion of enum to int or other data types will also need to be updated.

I'd also prefer removing the type prefix to the enum values as well: Button::Type::TOGGLE and Button::Type::NORMAL
A separate PR can be made to remove the ALL_CAPS naming.

As a side note, I did a search for enum definitions which were not enum class by using a negative look ahead regex:
Regular Expression in Atom Text Editor, exclude string “abc” from beginning of string
Find: enum (?!class)
Replace: enum class
The find is potentially more useful than the replace, since each changes results in many errors due to the name prefix requirement of values for enum class.

I would also like to do some renaming, though that was something I had planned to defer on.
I figure the rename and dropping ALL_CAPS can be done at the same time.

Why defer when you can do it all right now. You're already changing the point of use anyway.

Technically it only changes the ones that aren't already prefixed. Though that is likely to be all of them, or very nearly all of them.
Mostly though I just didn't want to think that deeply about the change, nor invest the time in renaming. I wanted something quick and mechanical that would let me know how widespread the changes would need to be. Turns out, quite a lot.
In terms of renaming the fields, if they are at least prefixed first, it makes search/replace of text safer later. Though with the prefix built into the old names, I suppose they're already pretty unique.
I was also considering changes that would make certain enums obsolete. For instance, it's not likely someone would change a Button between Normal and Toggle at runtime, after the Button was created. Perhaps it should be part of the type, with two different subclasses, rather than a parameter.
Still making decisions on where I want to go with some of these.

Consider installing Visual Assist for Visual Studio. It makes renaming much easier.

That's true. I do have at least temporary access to Visual Studio now.


branch out b4 PR

Request to submit testing changes. Let's see if it works

changes approved. thanks

changes approved. thanks


row detail does not work if the rows object is passed into a pipe

I'm submitting a ...  (check one with "x")
[x ] bug report => search github for a similar issue or PR before submitting
[ ] feature request
[ ] support request => Please do not submit support request here, post on Stackoverflow or Gitter

Current behavior
I have set up a detail row on my datatable and am applying a filter using a pipe i.e.
[rows]="myList | myfilter"
the detail row does not show when the row expand is triggered using
  "this.table.rowDetail.toggleExpandRow(row);"

Expected behavior
the rows list would be filtered by the pipe and the expanded row would correctly show when triggered.
Reproduction of the problem
Add a filter function as a pipe to existing row array binding.
What is the motivation / use case for changing the behavior?
using a filter to reduce the number of rows returned speeds up the grid, need to apply a detail section as the row contains too many columns
Please tell us about your environment:
Windows,  angular 4.3.6

Table version: 0.8.x
^10.2.3

Angular version: 2.0.x
4.3.6

Browser: [all | Chrome XX | Firefox XX | IE XX | Safari XX | Mobile Chrome XX | Android X.X Web Browser | iOS XX Safari | iOS XX UIWebView | iOS XX WKWebView ]
ALL

Language: [all | TypeScript X.X | ES6/7 | ES5]
Typescript 2.4

I was able to reproduce this exact issue today. I ended up taking the pipe out of the rows object for now.
Thanks for confirming this issue.

I ran into this problem today. Rows being filtered through a pipe can't be expanded (let-expanded is always false). Would love to see this addressed!

+1

Any progress / fix on this?

see issue #1612

I have it fixed by using the tmp repo suggested in #1612


Emit downlevel parameter initializers unless we are certain they don't have any side effects.

i.e. if you have:
function foo(a = "") { }
Then we can emit that as:
function foo(a) { }
but if you have:
function foo(a = bar()) { }
then we need to emit that as:
function foo(a) {
    if (a === void 0) { a = bar(); }
}

--
This also unifies some of emitting code between single line function blocks and other types of functions.

:+1:


Issuer Adapter Callback - Validate state and map token to userID

This was implemented as part of https://github.com/trustbloc/edge-sandbox/pull/407.


No  Alert View Controller Component?

Seems conspicuous by it's absence?
What do folks typically do when creating Apps built to Material guidelines for Alerts? Just use Apples UIAlertController? Can't believe they do as it's not customisable...

Sorry, I didn't see this issue until now. We have Dialogs coming out, courtesy of @OrkhanAlikhanov. We will be grooming it and adding in default Dialogs. This is internally noted. Thank you!


Compacted show options in General Tools tab

Its a very small change but saving space always good ^^

Can you try to get rid of that merge commit?


fix: compile contracts before generating docs

Motivation
While solving the issue #7331, I tested the forge doc on a sample contract.
Example to reproduce the issue:
// SPDX-License-Identifier: MIT
pragma solidity 0.8.24;

interface IERC20 {
    /**
    * @notice Mints `amount` token to `recipient`.
    * @param  recipient The address of the account receiving minted token.
    * @param  amount    The amount of token to mint.
    */
    function mint(address recipient, uint256 amount) external;
}

contract Test is IERC20 {
    /// @inheritdoc IERC20
    function mint(someType recipient, someType amount) external onlyOwner {}
}

Here you can some issues like someType is not a valid variable type and also onlyOwner isn't defined. But if we run forge doc here, it runs successfully without giving any errors and gives an output like this.
**Parameters**

|Name|Type|Description|
|----|----|-----------|
|`recipient`|`someType`|The address of the account receiving minted token.|
|`amount`|`someType`|   The amount of token to mint.|

You can see in the type someType comes which is not a valid solidity type.
Solution
compile contracts before running forge doc.
If we now run the forge doc, it will give an error of something like this:
[⠒] Compiling...
[⠢] Compiling 1 files with 0.8.24
[⠆] Solc 0.8.24 finished in 25.80ms
Error:
Compiler run failed:
Error (7920): Identifier not found or not unique.
  --> src/Test.sol:15:19:
   |
15 |     function mint(someType recipient, someType amount) external onlyOwner {}
   |                   ^^^^^^^^

lgtm

Hey @onbjerg! I have made this changes, just letting you know.
let compiler = ProjectCompiler::new(); --> let compiler = ProjectCompiler::new().quiet(true); to run the forge compile silently.

sgtm

This broke ignore option as it is not respected when the contract is compiled


Bug Report: Crash on copying to clipboard

Copying text to the clipboard is extremely flaky.
Environment
[Environment]::OSVersion

Platform ServicePack Version      VersionString
-------- ----------- -------      -------------
 Win32NT             10.0.18362.0 Microsoft Windows NT 10.0.18362.0

Windows Terminal (Preview)
Version: 0.5.2681.0

Steps to reproduce
Open up any console app and copy text via mouse clicks. OneDrive link to desktop recording via Radeon Software.
Expected behavior
Text be on clipboard and not crash.
Actual behavior
Sometimes the app crashes after ~0.5 seconds when trying to copy to the clipboard (content is lost), even if copying is successful, closing the app clears the clipboard. Copying via Ctrl+C is more reliable, but Ctrl+C is reserved for break.

This sounds close enough to the issues mentioned in https://github.com/microsoft/terminal/pull/2927 - so hopefully fixed with the next release.

Yeah. This'll be fixed by #2927. Thanks @Fractura for finding it.


Update layout.html to support a sphinx version that is not three-integers

Update layout.html to support a sphinx version that is not three-integers.
Useful for sphinx==5.2.0.post0, which would cause the theme to fail otherwise. Closes #1343.
Note that packaging is already a dependency transitively - but it is now an explicit dependency.
I added readthedocs-sphinx-ext as a test dependency, as I believe it is (and I couldn't get the tests passing locally without it).

By the way: I tested this with a theme that extends sphinx_rtd_theme, and it all checks out nicely - I am able to access sphinx_version_info from a custom layout, such as:
{% extends "sphinx_rtd_theme/layout.html" %}

{%- block htmltitle %}
{{ super() }}
Hello world
{{ sphinx_version_info  }}
{% endblock %}

Where the theme I created had a config like:
[theme]
inherit = sphinx_rtd_theme

And registering the theme with:
def config_inited_handler(app: Sphinx, config: Config) -> None:
    """
    Handler to register the theme's extra templates directory once the config
    has been read.

    """
    _ = app  # Unused.

    config.templates_path.append(str(HERE/'templates'))

def setup(app: Sphinx):
    """
    Configure the given Sphinx app to know about the acc_py theme.

    """
    app.add_html_theme('my-theme', str(HERE))
    # Register the theme's extra templates directory once the config has been
    # read.
    app.connect('config-inited', config_inited_handler)

Posting here to validate that adding the new use of app.connect("html-page-context", extend_html_context) shouldn't result in any unexpected surprises...

Posting here to validate that adding the new use of app.connect("html-page-context", extend_html_context) shouldn't result in any unexpected surprises...

This was also the one that caught my interest. But I think that it's safe. As this event is only relevant for the HTML builder, I believe that we can trust the output from the PR build.

Thanks so much @pelson for the solid work and addressing all the feedback :100:


Private hex repo support

This PR addresses https://github.com/dependabot/dependabot-core/issues/1286
Underneath, this required pretty minimal changes to the existing hex updater/checker code. Figuring out how to make the modifications and recognizing that all of the other tests made actual HTTP calls took most of the time.
Currently, it automatically fetches the public_key and doesn't perform any verification against a fingerprint. I opted out of adding public_key support initially because most repo consumers don't have direct access to the public_key anyhow, and they'd have to fetch it the same way.  It would be trivial to accept a public_key option or a fingerprint for verification.
A minimal server that has two versions of the jason package is now hosted on a free fly.io instance at dependabot-private.fly.dev. The hex server's source is here: https://github.com/sorentwo/dependabot-private-repo. (I'm happy to transfer the repository or the instance over to dependabot.)

Hi @sorentwo, thank you for this contribution, for including tests, and for linking the hex server source code and making it public! 🌟
I'm not a Hex expert, but from what I can tell these changes look okay. I posted some questions that will hopefully make this easier to review.
I wonder if there's a Hex package for verifying the integrity of public keys?
Since the changes and the Hex server source code check out, I'm going to approve and run CI for this branch. We should be able to get back to you on merging this later in the week ⌛

Thanks for taking a look!

I wonder if there's a Hex package for verifying the integrity of public keys?

There are libraries in erlang for doing the verification. I'll add another couple of scenarios to cover verifying the public key.

@Nishnha Ok, changes made 👍 The last three commits are reasonably descriptive and atomic. The biggest changes are in the last commit and I'll leave some comments inline.

@Nishnha The rubocop errors are all fixed up now and CI should pass on the next run.

@sorentwo Dependabot also has a separate step for updating lockfiles: https://github.com/dependabot/dependabot-core/blob/main/hex/lib/dependabot/hex/file_updater/lockfile_updater.rb
it uses the same organization_credentials as the check_update and get_latest_resolvable_versions methods did in the version resolver, and would probably need access to hex_credentials to properly support private repos.
I'm not sure where #update_lockfile_content itself is called from, but you might be able to just copy the run.exs file a la https://github.com/dependabot/dependabot-core/blob/main/hex/lib/dependabot/hex/file_updater/lockfile_updater.rb#L25 and swap out the organization_credentials for hex_credentials and have it all work.
I am also looking into any additional plumbing that we might need to flesh out internally to pass private hex repo credentials in from dependabot.yml following the schema in https://github.com/dependabot/dependabot-core/issues/1286#issuecomment-905696313

@Nishnha Great catch! I've made the suggested change, with a light refactoring to share the credential extraction logic.
Let me know if there's anything else I can do to support internal plumbing changes 🙂

@sorentwo thanks for your hard work! I took a look at the PR, just to try and get a sense of what you're doing, and everything looks great
@Nishnha I hope you don't mind the ping, but I wanted to bump this up to you! It's currently blocking my use of dependabot in our Elixir application :(

Hi @sorentwo and @cam-carter 👋🏾
We discussed prioritizing this issue during our team's sprint planning meeting today.
Unfortunately, we don't have the capacity for dedicated engineers to work on the internal-half of this issue this quarter. However, it is being tracked in our backlog, so we may get to it in-between planned project work.
The internal work mostly consists of writing a parser for the private repo schema that we would add to dependabot.yml and adding documentation on the feature.
The Elixir/Erlang/Hex ecosystem has been growing in importance across the GitHub security supply chain, so we hope to prioritize this--alongside other Hex ecosystem features--as planned project work on our roadmap, but we can't really provide a solid time estimate for when that might be.

Really hoping this one can be merged soon! 🤞

Thanks @sorentwo! Glad to see this being handled 🙂

Thank you as well @sorentwo, looking forward to seeing this PR merged soon! :smiley:

Hi @Nishnha, I hope you don't mind the ping! I was wondering if you might have a better idea on the timeline for this issue and the internal work that would be necessary by now? 🙏

@SophieDeBenedetto as an Elixirist inside of GitHub, any chance you could offer the dependabot team any insight into the internal work necessary to support this? Can the community be leveraged to assist if the dependabot team doesn't have any bandwidth to support what is necessary to get this working?

hello, anything new about this PR please?

@SophieDeBenedetto as an Elixirist inside of GitHub, any chance you could offer the dependabot team any insight into the internal work necessary to support this? Can the community be leveraged to assist if the dependabot team doesn't have any bandwidth to support what is necessary to get this working?

👋 hey @gjastrab! Sorry I completely missed this over the summer.

any chance you could offer the dependabot team any insight into the internal work necessary to support this

I'm not sure how to answer this question right now. It seems like you have a working solution in this PR so could you elaborate on what is left to be done that could benefit from additional support?

Can the community be leveraged to assist if the dependabot team doesn't have any bandwidth to support what is necessary to get this working

I'd be happy to help with this if we can flesh out what is needed a bit. One option would be to identify a working group through the Erlang Ecosystem Foundation, which I sit on the board of, and appeal for volunteers there. Another option might be identify some folks willing to take this on and apply for funding through the foundation. Otherwise, I can help put the word out for maintainers more generally around the community.
I'd be happy to set up a time to chat if you think that would be helpful. Let me know!

Supporting private hex repos would be hugely beneficial. We were really enjoying using dependabot at my company, but when we recently adopted Oban Pro, a few weeks later we realized dependabot was broken.
Unfortunately, it seems like there's no workaround if one of the dependencies is hosted on a private hex repo and Oban Pro is very popular, so this is affecting lots of customers. Dependabot has been a valuable tool to keep our dependencies up-to-date, so we'd love if this fix could be prioritized so we don't have to go back to manually monitoring our dependencies like we did pre-dependabot.
Thanks for building such a great tool and please let us know if there's anything we can do to help!

Hi again @sorentwo and @gjastrab, thanks for clarifying, I now understand that this PR is blocked by internal work that has yet to be sourced by GitHub's dependabot team. I don't think there's anything that the external community can do to move that work forward at this time, and I'm not aware of the dependabot team's priorities at this time. I'll try reaching out to see if that team is able to share any further info with me.

Couldn't agree more with @lleger!

Agreed! @SophieDeBenedetto can you see if the Dependabot team can prioritize this? It looks like most of the work has already been done.

We switched to Depfu solely because of this. Would be great to see this addressed so we can consider dependabot again.

👋 Hello from the Dependabot team! We're aware of this request, but haven't been able to dedicate engineering hours to it amidst a lot of other, long-standing, improvements to Dependabot the service.
We're going to figure out what needs to be done for this on our end tomorrow. I'm hesitant to share the results because I can't make any commitments, but I'll strive for some level of transparency. We do want to improve support for this community.

Separately, I want to ask that @SophieDeBenedetto not be pinged about this or other Dependabot issues 🙏 It's an awkward position to be put in. She's been an advocate for the Elixir/Erlang community and has escalated concerns to our team. However, getting this across the finish line is solely on the Dependabot team.

👋 @jurre and I paired for a bit to determine what needs to be done to make this change a reality. We've identified some steps that would need to be taken, but we need some more information before we can say how much work this will actually be.

🖼 Context
We believe a couple of changes will need to be made here. I'll list those below. First, it might help to give some context on how Dependabot makes authenticated requests in GitHub's production environments. We're careful to make sure that dependabot-core never has direct access to any credentials. Instead, all http(s) requests from dependabot-core are routed through a job-specific proxy server holds these credentials[^1]. The proxy server then injects the credentials for the registry that’s being accessed for the request.
✅ Tasks

Due to how credentials are handled in our production enviornments, the changes in this PR will not affect requests to private Hex registries. To get authentication right, we would need to implement this logic in the proxy server. The proxy server's source code is private, so this work will have to be done by the Dependabot team.

In order to pass along a public key fingerprint to the proxy, we would need to update the private registry schema for Hex. This should be relatively straight-forward. This will also need to be done by the Dependabot team.

🙏 Our ask
We're not clear on the HTTP contract for private Hex servers. We'll need to understand this better in order to update the proxy server. Ideally, this would be some kind of spec for private Hex servers, but short of that we could use documentation. We're hoping that you folks could provide us with an up-to-date spec that you can vouch for.
🕐 Timeline
I hope to not sound hand-wavy here, but the timeline will depend on what's involved in meeting the spec. When we have that, we can figure out what changes need to be made.
[^1]: The proxy server is short-lived, and only accessible for an individual job.

I haven't been actively working in Elixir for a while, and I don't have access to a project w/ oban_pro anymore, but here's some thoughts from cursory research I had done on this.
A good example of how private repos are used are contained in oban pro's installation docs.
Repo config docs are here.
I had done an initial bit of digging on this and while hex repos in dependabot-core today have hard-coded urls based on the package name, the bundler logic is more robust and I assume could be used as an example of how to implement it in hex. Bundler uses a bundler subprocess to lookup the private repos, and while those helpers don't exist for hex today, I'm sure they could be replicated for hex to use.
The dependency file parser would need to parse the repo that's listed w/ the hex package in the mix.lock, and pass it to get the url. The repo should be the second to last element of the tuple.
# mix.lock
 %{
   "oban_pro": {:hex, :oban_pro, "0.12.6", "[trunc]", [:list, :of, :deps], [], "oban", "[trunc]"}
   }

mix hex.repo show oban --url  will get you the url that the dependency should be pulled from.

@landongrindheim Thank you for looking into what it will take to complete this. If you need any additional details beyond the hexpm specifications that @BobbyMcWho linked above, please let me know.

📢 Update
I was able to spend some time on this last Friday and paired with @jurre today to wee where we could get. Still can't make any promises re: timeline, but we're much closer than we were 🎉

❓ In trying to reason through the private repositories spec and this PR, I think we understand the way Hex will handle these requests once this is in place. This package ecosystem seems to work differently from others that we handle, so I want to make sure. Does 👇 sound accurate?

Make an unauthenticated request to the private registry's /public_key endpoint
Verify the validity of the private registry (public key/fingerprint comparisons)

Hex seems to handle the details once we've compared the public key/fingerprint

Make authenticated requests against the verified private registry

We'd like to test our internal implementation before shipping this change. We don't have a private Hex server. Are we able to test against the instance listed in the description of this PR?

This package ecosystem seems to work differently from others that we handle, so I want to make sure. Does 👇 sound accurate?

That’s accurate as I understand it. There is an exception if the “skip key validation” option is used, then it doesn’t compare the signature to the private key.

Are we able to test against the instance listed in the description of this PR?

Absolutely! That’s precisely why I built and deployed it.

@sorentwo I've been trying to set up a test repository using the private Hex server instance you shared. I forked the repo for the service and deployed it, but have so far been unable to get the dependencies declared properly for Dependabot to update.
I've bypassed the trust mechanisms to get jason set up in a mix.exs/mix.lock, but get the following response:
Request failed (404)
** (Mix) Package fetch failed and no cached copy available ({{url}}/tarballs/jason-1.1.0.tar)

Could you set up a repo that we could fork and have your private Hex server instance running so we can do some tests internally?

One more question: does the request to /public_key typically require authentication? It seems to on the shared instance, but I wouldn't have expected it because it's a public key. If it does require auth, we'll need to account for that internally, too.

Could you set up a repo that we could fork and have your private Hex server instance running so we can do some tests internally?

You forked the correct repository, I just hadn't committed the tarballs! Apparently, .tar files are ignored by my global .gitignore and I missed them. I've force-committed those two files now and it everything should work.
The https://dependabot-private.fly.dev/ instance is still running, if you'd like to point to that in the meantime.

One more question: does the request to /public_key typically require authentication? It seems to on the shared instance, but I wouldn't have expected it because it's a public key. If it does require auth, we'll need to account for that internally, too.

Oban Pro's private registry and the example app I wrote do require authentication for the public_key, but there isn't a standard that dictates it. If it simplifies your internal flow I'm happy to bypass auth for the /public_key.

@sorentwo Looks like the test instance still requires authentication for the /public_key endpoint:
1) Dependabot::Hex::UpdateChecker#latest_resolvable_version with a dependency from a private repo with dependencies on both a private organization and private repo
[185](https://github.com/dependabot/dependabot-core/actions/runs/3431316177/jobs/5719307391#step:6:186)
     Failure/Error: raise Dependabot::PrivateSourceAuthenticationFailure, repo
[186](https://github.com/dependabot/dependabot-core/actions/runs/3431316177/jobs/5719307391#step:6:187)

[187](https://github.com/dependabot/dependabot-core/actions/runs/3431316177/jobs/5719307391#step:6:188)
     Dependabot::PrivateSourceAuthenticationFailure:
[188](https://github.com/dependabot/dependabot-core/actions/runs/3431316177/jobs/5719307391#step:6:189)
       The following source could not be reached as it requires authentication (and any provided details were invalid or lacked the required permissions): dependabot
[189](https://github.com/dependabot/dependabot-core/actions/runs/3431316177/jobs/5719307391#step:6:190)
     # ./lib/dependabot/hex/update_checker/version_resolver.rb:81:in `handle_hex_errors'
[190](https://github.com/dependabot/dependabot-core/actions/runs/3431316177/jobs/5719307391#step:6:191)
     # ./lib/dependabot/hex/update_checker/version_resolver.rb:48:in `rescue in fetch_latest_resolvable_version'
[191](https://github.com/dependabot/dependabot-core/actions/runs/3431316177/jobs/5719307391#step:6:192)
     # ./lib/dependabot/hex/update_checker/version_resolver.rb:32:in `fetch_latest_resolvable_version'
[192](https://github.com/dependabot/dependabot-core/actions/runs/3431316177/jobs/5719307391#step:6:193)
     # ./lib/dependabot/hex/update_checker/version_resolver.rb:24:in `latest_resolvable_version'
[193](https://github.com/dependabot/dependabot-core/actions/runs/3431316177/jobs/5719307391#step:6:194)
     # ./lib/dependabot/hex/update_checker.rb:177:in `fetch_latest_resolvable_version'
[194](https://github.com/dependabot/dependabot-core/actions/runs/3431316177/jobs/5719307391#step:6:195)
     # ./lib/dependabot/hex/update_checker.rb:32:in `latest_resolvable_version'
[195](https://github.com/dependabot/dependabot-core/actions/runs/3431316177/jobs/5719307391#step:6:196)
     # ./spec/dependabot/hex/update_checker_spec.rb:219:in `block (3 levels) in <top (required)>'
[196](https://github.com/dependabot/dependabot-core/actions/runs/3431316177/jobs/5719307391#step:6:197)
     # ./spec/dependabot/hex/update_checker_spec.rb:456:in `block (5 levels) in <top (required)>'
[197](https://github.com/dependabot/dependabot-core/actions/runs/3431316177/jobs/5719307391#step:6:198)
     # /home/dependabot/dependabot-core/common/spec/spec_helper.rb:60:in `block (2 levels) in <top (required)>'
[198](https://github.com/dependabot/dependabot-core/actions/runs/3431316177/jobs/5719307391#step:6:199)
     # ./.bundle/ruby/3.1.0/gems/webmock-3.18.1/lib/webmock/rspec.rb:37:in `block (2 levels) in <top (required)>'
[199](https://github.com/dependabot/dependabot-core/actions/runs/3431316177/jobs/5719307391#step:6:200)
     # ------------------
[200](https://github.com/dependabot/dependabot-core/actions/runs/3431316177/jobs/5719307391#step:6:201)
     # --- Caused by: ---
[201](https://github.com/dependabot/dependabot-core/actions/runs/3431316177/jobs/5719307391#step:6:202)
     # Dependabot::SharedHelpers::HelperSubprocessFailed:
[202](https://github.com/dependabot/dependabot-core/actions/runs/3431316177/jobs/5719307391#step:6:203)
     #   Downloading public key for repo "dependabot" failed with code: 502
[203](https://github.com/dependabot/dependabot-core/actions/runs/3431316177/jobs/5719307391#step:6:204)
     #   /home/dependabot/dependabot-core/common/lib/dependabot/shared_helpers.rb:122:in `run_helper_subprocess'
[204](https://github.com/dependabot/dependabot-core/actions/runs/3431316177/jobs/5719307391#step:6:205)

We should probably find a better solution for testing this long-term (one that doesn't rely on a contributor's resource), but for now could you relax authentication on that server as well?

Looks like the test instance still requires authentication for the /public_key endpoint

Oops, I missed that one. Fixed with https://github.com/sorentwo/dependabot-private-repo/commit/919b158fb27827f80ae1212c467e9e5ba9ae06d5 and deploying now.

I've been testing these changes along with some internal changes and things are looking good 🎉 This is an awesome addition, and I'm sorry it's taken us so long to get it landed. I hope we can celebrate this contribution and make it known to the Elixir/Erlang community 😄

🚶 Next Steps

Get CI 🟢
Update the branch

You can merge or rebase. Feel free to squash commits (including 919f469 I made with a lousy message 🤦)

Another round of testing

If you want to test this with Oban Pro, we can coordinate and do some testing in a private repo 🙂

🚢

👷 My tasks

Update the docs to include the new registry listing
Make sure everything is in place for this to be shared

🕐 Timeline

Tomorrow is a US holiday, so I'll be out. I'm hoping we can get this fully tested next week and running in production. We probably won't announce it until it lands in our docs, but I'm hopeful that we can have some folks testing/using it before then 🤞

[x] Get CI 🟢 (last "hexpm" check fixed it)
[x] Update the branch 👍 (rebased)
[ ] Another round of testing (all you!)

If you want to test this with Oban Pro, we can coordinate and do some testing in a private repo 🙂

That sounds fantastic. I can offer up our private repo that powers getoban.pro 👍

As an Oban Pro user, I appreciate everyone's work on this PR! If I can help test, please let me know.

🎉 It's alive!
Check out the Changelog post for the announcement. Thanks to @sorentwo for this fantastic contribution!
Please reference our updates docs for configuring Dependabot to update your privately-hosted Hex packages 😄

Did anyone manage to get the new feature working? We haven't, this is what we have:
{:oban_pro, "~> 0.12", repo: "oban"}

and
version: 2
registries:
  github-hex-repository:
    type: hex-repository
    repo: oban
    url: https://getoban.pro/repo
    auth-key: ${{ secrets.OBAN_PRO_LICENSE_KEY }}
    public-key-fingerprint: "SHA256:xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx"
updates:
  - package-ecosystem: "mix"
    directory: "/"
    registries:
      - github-hex-repository
    schedule:
      interval: daily

Any idea? (we've added the correct dependabot secret called OBAN_PRO_LICENSE_KEY) Is there a way to see some logs of what went wrong?

yes, you can see logs in Insights. But you have to add
insecure-external-code-execution: allow

for example in your config:
version: 2
registries:
  github-hex-repository:
    type: hex-repository
    repo: oban
    url: https://getoban.pro/repo
    auth-key: ${{ secrets.OBAN_PRO_LICENSE_KEY }}
    public-key-fingerprint: "SHA256:xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx"
updates:
  - package-ecosystem: "mix"
    directory: "/"
    insecure-external-code-execution: allow
    registries:
      - github-hex-repository
    schedule:
      interval: daily

@oliver-kriska awesome, thank you so much, didn't know about that page! That explains a lot :)


It is incredibly non-obvious to start a NEW drawing, why ? `

Show me where "New" is. I have been doodling and I want to make a new drawing and every time I try and quickly do this I get annoyed and just do it on paper.
I know I'm being dense, but I can't help but feel this is something lots of people look for.
I'm sure its here - but the UX is NOT obvious.

@WillowHQ  press Ctrl+Delete to clear everything, or click on the right sidebar and select 'Reset Canvas'.

https://github.com/user-attachments/assets/b4b5bc7a-2b80-4ab9-b37d-07bafd8933b2
Following up from @imran1khan - @WillowHQ Hopefully this clears up that functionality here.
As a discussion point - perhaps there's an improvement for clarity.

Do you have any suggestions?
What did you try to do to acheive this that wasn't working? Perhaps we could look there to expose the reset functionality

I put up a PR that exposes this command shortcut within the Main menu - perhaps that is sufficient here?

@WillowHQ press Ctrl+Delete to clear everything, or click on the right sidebar and select 'Reset Canvas'.

I've found that this fails to clear all the data for the current canvas as expected.
For example, I was collaborating in a shared session today, the "Canvas is too big/couldn't save to backend" error message appeared, so I saved the board and cleared the canvas with cmd+delete to try to resolve the error.
This did not remove the error, and the performance of the canvas continued to be laggy as if the canvas was still loaded with objects, even though no objects were visible.
I tested this by drawing on a board, doing cmd+delete, then cmd+Z. You're able to undo changes before the canvas reset and view previous states of that canvas. If it were a new canvas, this wouldn't be possible.
This presents a performance issue for one, and possibly a data containment issue, since users might expect that the data is permanently gone after clearing. This expectation is especially reinforced by the "This will clear the whole canvas. Are you sure?" dialog box. There is no other alternative feature on the front end that allows the actual deletion of the underlying data.
I'm assuming that this data can be removed by deleting the local storage during single user sessions, but I assume an error like "couldn't save to backend" implies your data is being stored on a server for live collaboration sessions.
Having clear delineation between delete this canvas entirely and start a new one and erase all objects within this canvas (but keep this canvas) is critical, both for convenience and for clarity about what data exists where.


add loongarch Architecture for bn

Signed-off-by: lujinfeng<EMAIL_ADDRESS>
add  loongarch-mont.pl writed by loongarch , can improve the performance of “speed rsa” on loongson machine。
While 512-bit RSA sign operations can be 65.7% faster in 64-bit mode, 1024-bit ones are 24.6% faster, and 2048-bit ones are only 10.4% faster, the code improve rsa512 verify benchmark by 20.4%, rsa1024 one - by 10.9%:-)
Checklist

[ ] documentation is added or updated
[ ] tests are added or updated

i have CLA , but sending fails with the following email address.
<EMAIL_ADDRESS>

i have CLA , but sending fails with the following email address<EMAIL_ADDRESS>
Please resend it now. We apologize for having the mail server down temporarily.

I have sent it successfully. Please check it. My email address is<EMAIL_ADDRESS>

@wannaNo, you sent the CCLA, we also need an ICLA from you.


Function of times_per_scene

Hi,
Thank you for the work. Could you please clarify the meaning behind "times_per_scene" in dataset_re10k.py?
Thanks in advance

Hi @Shubhendu-Jena, it is just an implementation hack to support testing DTU (see here) with the RE10K data loader. It aims to test the same scene several times with different context and target views. When dealing with the RE10K and ACID datasets, it should always be set to times_per_scene=1.


Inquiry for something complicated.

This might be a bit complicated for this plugin but I figured I'd ask. Would it be possible to work this in with the BossLands plugin? I want to make an NPC at a guildhall in my spawn town who give you info on the bosses from that plugin. I want to make dialogue where he says where to go to kill the bosses in order starting with the first boss. Once you kill the first boss you return to the NPC for a reward and then the NPC will give you the info for the next boss. Rinse Repeat until all bosses are defeated. Would that be doable? I have npcs on my server using Citizens 2 and DTL Traders.

I'm trying out this plugin and am completely lost. The vid is outdated and all I want to do is pretty much the same pig kill/deliver pork quest in the tutorial vid to learn it and try it out but everything is different and I don't quite get the wiki. any tips? :D

Is this the plugin you're referring to?
https://www.spigotmc.org/resources/boss-land.68320/
For dialogue, you can put any text you like for the ask message. If you meant to ask for a custom objective for killing bosses from that plugin, it may be possible if the author provides an API. I don't see any mention of one from the plugin page and the source code doesn't appear to be available.
I would recommend contacting the author about whether an API exists. If so, the fastest way to get a module created for you is to visit the hiring forum (not affiliated with Quests) and direct your developer to the following page:
https://github.com/PikaMug/Quests/wiki/Master-‐-Custom-Quest-API#objectives-api

Yep that's it :D oof that's a whole lotta complicated-ness. How about the other thing I posted below? I'm totally lost on even how to use this plugin and it seemed some parts weren't working like when i put an action to kill 3 pigs it didn't apply and i see no option to deliver items etc. It's completely different than what the Video shows on the page and I'm completely new to making MC stuff/servers so I'm totally lost. I found the wiki a little difficult to understand to. Any ideas or tips? I just want to replicate the pig quest from the video but add a delivery step where u go back to the npc and give it 3 pork to complete the quest. Thanks :D

@MrRayhonda Please limit inquiries to one per ticket. I’ll try to answer your question, but do open a separate ticket if you need further assistance.
With this plugin, there are two major prompts to know about: /quests editor and /quests actions. You want to be using the first one.
Each quest can have multiple stages, which are credit using the Edit Stages option after you’ve named your new quest. Once you’ve created a stage, you may have multiple Objectives.
The “Kill mobs” objective can be used to require your players the kill monsters or animals, like pigs. The “Deliver items” objective (under the “NPC” submenu in newer versions) requires your player to deliver an item to a Citizens NPC. When adding an item to the objective, try holding a porkchop in your hand and selecting option 0.
Sent with GitHawk


why ws4w can't use Ethernet to share network

My pc have wifi and ethernet. I want to enable ICS for ethernet, but ICS is allways be enabled for wifi.
Is it my system's issue?

Hi @RogerSugit, when you select the option to Enable Internet Sharing, you are presented with a list of all of your network adapters, and you can choose the one you want to share. Are you saying that you pick the ethernet adapter from the list, but the Wi-Fi adapter's connection is the one that's shared?
If so, you may need to look in Windows. Open Network Connections in the Control Panel. Right-click > Properties on the Wi-Fi adapter, go to the Sharing tab, and ensure that "Allow other network users to connect through this computer's Internet connection" is unchecked. Then go to the ethernet adapter and ensure the same option is checked. Unfortunately I've found sometimes the only way to enable this is through Windows.

Yes, I picked the ethernet adapter to share, but the Wi-Fi adapter's connection is the one that's shared.
The solution you said I have tried for many times, but it didn't work, ICS is always working on WiFi adapter.

Yes, I picked the ethernet adapter to share, but the Wi-Fi adapter's connection is the one that's shared.
The solution you said I have tried for many times, but it didn't work, ICS is always working on WiFi adapter.

Yes, I picked the ethernet adapter to share, but the Wi-Fi adapter's connection is the one that's shared.
The solution you said I have tried for many times, but it didn't work, ICS is always working on WiFi adapter.


adc example issue

I am trying to run this adc example. but I found this line of code failed to compile.
let mut adc = adc::Adc::adc1(p.ADC1, &mut rcc.apb2, clocks);

The error message is
error[E0061]: this function takes 2 parameters but 3 parameters were supplied
  --> src/main.rs:28:19
   |
28 |     let mut adc = adc::Adc::adc1(p.ADC1, &mut rcc.apb2, clocks);
   |                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ expected 2 parameters

error: aborting due to previous error

For more information about this error, try `rustc --explain E0061`.
error: Could not compile `blue-pill-quickstart`.

It seems the api has been changed.

Yes, that change was made in #67 which has not yet been published to crates.io which is why the example fails to build.
Until we publish it, you can have a look at the examples in the 0.3.0 tag https://github.com/stm32-rs/stm32f1xx-hal/tree/v0.3.0

Yes, that change was made in #67 which has not yet been published to crates.io which is why the example fails to build.
Until we publish it, you can have a look at the examples in the 0.3.0 tag https://github.com/stm32-rs/stm32f1xx-hal/tree/v0.3.0

Thanks, I guess I can use the github to directly use the current master branch rather than use the one in crates.io.

That works as well. I'll go ahead and close this issue since it will be fixed when we publish the next version


Ft hodl quest

[ ] issue #132
[*] code change includes tests

warning: Plugin diagnostic: Failed to generate ABI: Got unexpected type.
what could this issue with github action test @b-j-roberts

warning: Plugin diagnostic: Failed to generate ABI: Got unexpected type. what could the issue be with github action test @b-j-roberts

Not 100%, if building works locally, try rebasing/merging main. The format for quest events changed in main just now, so I'm assuming thats the issue here.

warning: Plugin diagnostic: Failed to generate ABI: Got unexpected type. what could the issue be with github action test @b-j-roberts

Not 100%, if building works locally, try rebasing/merging main. The format for quest events changed in main just now, so I'm assuming thats the issue here.

Alright

warning: Plugin diagnostic: Failed to generate ABI: Got unexpected type. what could the issue be with github action test @b-j-roberts

Not 100%, if building works locally, try rebasing/merging main. The format for quest events changed in main just now, so I'm assuming thats the issue here.

Alright

warning: Plugin diagnostic: Failed to generate ABI: Got unexpected type. what could the issue be with github action test @b-j-roberts

Not 100% sure, if building works locally, try rebasing/merging main. The format for quest events changed in main just now, so I'm assuming thats the issue here.

You are right, after rebasing/merging main into my branch, the error was fixed

@b-j-roberts kindly review the changes

inside art_peace.cairo
```

// TODO: Dev only - remove
let test_address = starknet::contract_address_const::<
0x328ced46664355fc4b885ae7011af202313056a7e3d44827fb24c9d3206aaa0
>();
self.extra_pixels.write(test_address, 1000); // line 162

this is make our test fail, if we set a different value, do i need to change the above line @b-j-roberts

this is make our test fail, if we set a different value, do i need to change the above line @b-j-roberts

This line shouldn't effect the tests because it is a different address than PLAYER1

@b-j-roberts kindly this PR, thank you

@b-j-roberts kindly review

from this code
        fn is_claimable(
            self: @ContractState, user: ContractAddress, calldata: Span<felt252>
        ) -> bool {
            if self.claimed.read(user) {
                return false;
            }

            let art_peace_main = IArtPeaceDispatcher { contract_address: self.art_peace.read() };
            println!("user call this function - is_claimable: {:?}", user);
            let get_extra_pixels_count = art_peace_main.get_user_extra_pixels_count(user.into());
            println!("user extra pixel - is_claimable: {:?}", get_extra_pixels_count);

            println!("caller pixel count: {:?}", art_peace_main.get_extra_pixels_count());

            println!("user reward - is_claimable: {:?}", self.extra_pixel.read());

            if get_extra_pixels_count >= self.extra_pixel.read() {
                return true;
            }

            false
        }

user extra pixel - is_claimable: 0
caller pixel count: 0
user reward - is_claimable: 10
which mean both art_peace_main.get_extra_pixels_count() and art_peace_main.get_user_extra_pixels_count(user.into() are returning 0 @b-j-roberts

and from this code
 fn claim_main_quest(ref self: ContractState, quest_id: u32, calldata: Span<felt252>) {
           let now = starknet::get_block_timestamp();
           assert(now <= self.end_time.read(), 'ArtPeace game has ended');
           let quest = self.main_quests.read(quest_id);
           let user = starknet::get_caller_address();
          //  println!("user call this function - claim_main_quest: {:?}", user);
           let reward = IQuestDispatcher { contract_address: quest }.claim(user, calldata);
          //  println!("user reward: {}", reward);

           if reward > 0 {
               self
                   .extra_pixels
                   .write(
                       starknet::get_caller_address(),
                       self.extra_pixels.read(starknet::get_caller_address()) + reward
                   );
               println!(
               "extra pixel after reward: {}",
               self.extra_pixels.read(starknet::get_caller_address())
           );
           }
           self.emit(MainQuestClaimed { quest_id, user, reward, calldata });
       }

we  are getting the following
extra pixel after reward: 18
user call this function - get_user_extra_pixels_count:<PHONE_NUMBER>1171505

this mean alone the path of executing claim_main_quest the user which is let user = starknet::get_caller_address(); is the same with  the user contract caller here

 fn get_user_extra_pixels_count(self: @ContractState, user: ContractAddress) -> u32 {
            println!("user call this function - get_user_extra_pixels_count: {:?}", user);
           //  println!("user extra pixel: {:?}", self.extra_pixels.read(user));
            self.extra_pixels.read(user)
        }

user call this function - get_user_extra_pixels_count:<PHONE_NUMBER>1171505

actually the log is showing the are the same, but
 self.extra_pixels.read(user) keep returning 0  @b-j-roberts

snf::store should be setup like this
snf::store(
        art_peace_contract,
        snf::map_entry_address(
            selector!("extra_pixels"), array![users address].span()
        ),
        array![value].span()
      );

It looks like you're using player 1 as the user for get_extra_pixels_count, but that's not reflected in your snf::store call. Value also needs to be the amount of extra pixels to set for the user.

snf::store should be setup like this
snf::store(
        art_peace_contract,
        snf::map_entry_address(
            selector!("extra_pixels"), array![users address].span()
        ),
        array![value].span()
      );

It looks like you're using player 1 as the user for get_extra_pixels_count, but that's not reflected in your snf::store call. Value also needs to be the amount of extra pixels to set for the user.

what exactly is this expected to do, update this
extra_pixels: LegacyMap::<ContractAddress, u32>,
or  this
    #[storage]
    struct Storage {
        art_peace: ContractAddress,
        reward: u32,
        extra_pixel: u32,
        claimed: LegacyMap<ContractAddress, bool>,
    }

snf::store should be setup like this
snf::store(
        art_peace_contract,
        snf::map_entry_address(
            selector!("extra_pixels"), array![users address].span()
        ),
        array![value].span()
      );

It looks like you're using player 1 as the user for get_extra_pixels_count, but that's not reflected in your snf::store call. Value also needs to be the amount of extra pixels to set for the user.

how can get the address of the caller  because this setup keep returning 0
  snf::store(
        art_peace.contract_address,
        snf::map_entry_address(selector!("extra_pixels"), array![(contract_address_const::<1>()).into()].span()),
        array![18].span()
    );

when i try this
 let get_extra_pixels_count = art_peace_main.get_user_extra_pixels_count(user.into());

 println!("user extra pixel - is_claimable: {:?}", get_extra_pixels_count);

is their anything wrong the contract itself, @b-j-roberts


Introducing a MigratedDataSource interface for easier integration

Which version and edition of Flyway are you using?
7.7
Which client are you using? (Command-line, Java API, Maven plugin, Gradle plugin)
Java API
Which database are you using? (Type & version)
PostgreSQL
Which operating system are you using?
Ubuntu Linux 20.04
This is a feature request. I'll be happy to provide a PR if it seems interesting to the development team
The currently recommended way to use Flyway in a Spring context is to rely on dependsOn (https://flywaydb.org/documentation/usage/api/#spring-configuration):
Example from doc:
<bean id="sessionFactory" class="..." depends-on="flyway">
While it works, it's sometimes a bit cumbersome and error-prone, especially when doing some config overrides for testing or other purposes. Sometimes a dependsOn just slips out of the way, and everything breaks without an apparent reason, since it tries to work with an non-migrated database.
My idea would be to introduce a marker interface, e.g. MigratedDataSource which just extends Java's DataSource interface, along with a default implementation which just delegates to the original datasource. Then, all beans depending on flyway being run before them can just rely on such type. The idea would be something like this (pseudojava) for a Spring java-based context:
@Bean
public MigratedDataSource flyway(DataSource ds) {
          ...
          Flyway.configure().datasource(ds)
          ...
          flyway.migrate()
          return new DelegatingDataSource(ds)
}

let me know if you think the idea interesting and useful. I know this can be done entirely outside Flyway (and I have done something similar already in some projects), but I think it would be useful to have it within the library.

Hey @alanfranz ,
We don't maintain integrations in this project, but instead aim to make our API as interoperable as possible,
however if this gains more traction then we'll consider it, but we'll close it for now.


Fix word-wrap on mobile devices

Reported by @DJ-Gordon.
When possible can somebody take a look at the line breaking for the links on the main page. It does not look great on Mobile when “OrgBook” break.

Fixed, tested in prod.  Users will have to refresh the page if they have visited the site previously.

@DJ-Gordon, could you retest and confirm this is fixed, please?

Yes. Looks much better.
[cid:3407F993-E24A-4750-A6A7-663A189A50AF-L0-001]
Thanks for the quick fix.
D
Sent from my iPhone
On Dec 27, 2021, at 8:04 AM, Wade Barnes @.***> wrote:

[EXTERNAL] This email came from an external source. Only open attachments or links that you are expecting from a known sender.
@DJ-Gordonhttps://can01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FDJ-Gordon&data=04|01|Dwayne.Gordon%40gov.bc.ca|e6204283db134f69551e08d9c95275be|6fdb52003d0d4a8ab036d3685e359adc|0|0|637762178840231098|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=4soLEfOBxKqsQANih%2FgaDbgOcT6uqVsHbyfzlM26QoY%3D&reserved=0, could you retest and confirm this is fixed, please?
—
Reply to this email directly, view it on GitHubhttps://can01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fbcgov%2Forgbook-bc-client%2Fissues%2F106%23issuecomment-1001634350&data=04|01|Dwayne.Gordon%40gov.bc.ca|e6204283db134f69551e08d9c95275be|6fdb52003d0d4a8ab036d3685e359adc|0|0|637762178840231098|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=HhBi%2BJvAvK51ejnIdk2OpsRbku31jkuAYmzavifq5q0%3D&reserved=0, or unsubscribehttps://can01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAEWHWFRS62ZZ2BN33NBEJ6TUTCE6BANCNFSM5KVNZC7Q&data=04|01|Dwayne.Gordon%40gov.bc.ca|e6204283db134f69551e08d9c95275be|6fdb52003d0d4a8ab036d3685e359adc|0|0|637762178840231098|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=D1rcO7PD0HrpAM2f3s1p5K8l%2BFoAr5lz7QwIkWUZUC4%3D&reserved=0.
Triage notifications on the go with GitHub Mobile for iOShttps://can01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fapps.apple.com%2Fapp%2Fapple-store%2Fid1477376905%3Fct%3Dnotification-email%26mt%3D8%26pt%3D524675&data=04|01|Dwayne.Gordon%40gov.bc.ca|e6204283db134f69551e08d9c95275be|6fdb52003d0d4a8ab036d3685e359adc|0|0|637762178840231098|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=CxdJn30p5N1ZxnOyQzdPxrag4q9pYUUWGbWYLW4Eedk%3D&reserved=0 or Androidhttps://can01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fplay.google.com%2Fstore%2Fapps%2Fdetails%3Fid%3Dcom.github.android%26referrer%3Dutm_campaign%253Dnotification-email%2526utm_medium%253Demail%2526utm_source%253Dgithub&data=04|01|Dwayne.Gordon%40gov.bc.ca|e6204283db134f69551e08d9c95275be|6fdb52003d0d4a8ab036d3685e359adc|0|0|637762178840231098|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=Q9QhDqi1DkhFkvzfFEIVOfkNdyFdS5rQ7uuTdurlnRk%3D&reserved=0.
You are receiving this because you were mentioned.Message ID: @.***>


Flaky ACME integration test

A rare error recently happened in ACME integration test, but the same test passes with re-run.
The reason of this flaky test might be the underlying MockFetcher.

The ACME client waits for 1 second before checking the status of HTTP challenge. However, MockServer expects new request comes within 1 second.
When the operating system lets the client thread to sleep more than 1 second, the integration test fails.
This can be fixed by increasing MockServer time limit.


basic auth credentials for api

Codecov Report

Merging #1341 (8ef6398) into main (455b8be) will decrease coverage by 0.04%.
The diff coverage is 65.00%.

@@            Coverage Diff             @@
##             main    #1341      +/-   ##
==========================================
- Coverage   62.48%   62.44%   -0.05%
==========================================
  Files         113      113
  Lines       12020    12027       +7
==========================================
- Hits         7511     7510       -1
- Misses       3915     3921       +6
- Partials      594      596       +2

Impacted Files
Coverage Δ

internal/conf/conf.go
73.70% <ø> (ø)

internal/core/formatprocessor_h264.go
75.43% <ø> (ø)

internal/core/hls_muxer.go
54.30% <ø> (ø)

internal/core/rtmp_conn.go
63.40% <ø> (ø)

internal/core/rtmp_source.go
65.06% <ø> (-2.74%)
:arrow_down:

internal/core/webrtc_conn.go
0.00% <ø> (ø)

internal/hls/client_processor_fmp4.go
65.89% <ø> (ø)

internal/hls/client_processor_mpegts.go
61.24% <ø> (ø)

internal/hls/fmp4/init.go
60.09% <ø> (ø)

internal/hls/fmp4/init_track.go
67.79% <ø> (ø)

... and 13 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Hello there are two possible ways to protect the API listener:

not protecting the API at all, add a reverse proxy before it and let the reverse proxy handle the authentication.
protecting the API, but i think that basic auth will not be enough for a lot of users that will ask for more advanced authentication techniques (separate credentials / JWT / external authentication)

We can certainly work towards the second approach, but keep in mind that handling security directly is not a "best practice" approach since there's a high risk of causing security issues. In my work i always try to avoid developing security components directly, i generally use a series of rugged security components (identity servers, gateways, WFAs, etc) and i put them before my services.
Anyway, we can certainly add a basic security layer to the server too, but i think that basic authentication will not be enough.

having a simple level of protection like basic auth would be useful, it would avoid adding additional software such as reverse proxy and its maintenance will still remain almost zero cost since everything is implemented by gin

@eravellaSC i'm working on an authentication system that will support explicit credentials, external authentication or JWTs along the entire server, including paths and API.
In the meanwhile, in order to protect the API, you can use a reverse proxy.

Hi @aler9 any update on this.

the implementation of jwt seems like something that is very time consuming, many people simply want to block requests to the api via basic auth, this solution however remains very simple to maintain and solves many of the cases

replaced by #3081


Strange caching-related errors

I use hot reloading and this plugin, and every so often I get an error like this after an update:
Uncaught TypeError: Cannot call a class as a function
    at _classCallCheck (webpack:///./~/redux-form/es/Field.js?:12:99)
    at Field (webpack:///./~/redux-form/es/Field.js?:39:7)

It always happened with redux-form so far. When I trace through the problem, there is a mixup of exports. The webpack_requires are supposed to import one export, but in fact return another export of the same module.

Note how it passes export b, hoping for reduxForm, but in fact getting Field
I don't change the redux-form code, so it's quite mysterious as to why it suddenly changes export.
The only way to make it go away is to clear the hard-source-webpack-plugin cache.

@wmertens Thanks for opening this issue!
I'm going to guess this is a bug related to https://github.com/mzgoddard/hard-source-webpack-plugin/blob/d76d65df0089bc93fb3df5f00f1af9e0d11c58ed/index.js#L623-L637. That part invalidates cached modules if their used and usedExports change. What I'm guessing is happening is you have two (or more) modules depending on a common dependency changing the usedExports or that common dependency is updated. Either way the common dependency is rebuilt and the order of usedExports may be different depending on the order of the modules as part of webpack considers them to build usedExports.
The short of this is to fix this I believe I need to also invalidate any reasons (webpack's name for dependents) or ensure the existing indices is preserved for usedExports. Either of those solutions has the potential for instability so I'm going to think about it a little to make sure I don't cause infinite rebuilds or bad builds.


Foreign born population

Description: The number and percent of people who are foreign born for three time periods
Logic:
Source(s):
Year(s):

[ ]  2000
[ ] 2008-2012
[ ] 2015-2019

Geographies:

[ ] PUMA
[ ] Borough
[ ] City

Race and ethnicity breakdown: Yes
Denominator: Total pop
SME: Population
Questions:
Roadblocks:

2008-2012 PUMS issue

We download this information in the SCHL PUMS variable. One of small number of variables we currently download and don't aggregate on


Updating parent after children update

I'm trying to implement a pretty simple model: (Mission --> MissionPage --> PageElement --> Options)
Where each parent has many children (One mission has many pages, each page has many elements, each element has many options).
I'm currently facing an issue when I update an option, when I query the option from its parent, I get the old instance (as it was not updated). I'm using a ForeignKeyContainer and trying to set the "saveForeignKeyModel = true" is giving me the following error
Cannot call save() on a foreign key container. Call load() and then save() instead
This is thrown at the Option table, which contains the following (among other methods):
Option Model:
@Table(database = DatabaseModule.class)
public class PageElementOption extends BaseModel {

@SerializedName("id")
@Column
@PrimaryKey
public long id;

@ForeignKey(saveForeignKeyModel = true)
ForeignKeyContainer<PageElement> elementContainer;

public void associateElement(PageElement element) {
    elementContainer = FlowManager.getContainerAdapter(PageElement.class).toForeignKeyContainer(element);
}

PageElement model
@ModelContainer
@Table(database = DatabaseModule.class)
public class PageElement extends BaseModel {

@PrimaryKey
@SerializedName("id")
@Column
public long id;

    @SerializedName("page_element_options")
@Expose
public List<PageElementOption> pageElementOptions;

@ForeignKey(saveForeignKeyModel = true)
ForeignKeyContainer<MissionPage> pageContainer;

@OneToMany(methods = {OneToMany.Method.ALL}, variableName = "pageElementOptions")
public List<PageElementOption> getPageElementOptions() {
    if (pageElementOptions == null || pageElementOptions.isEmpty()) {
        pageElementOptions = SQLite.select()
                .from(PageElementOption.class)
                .where(PageElementOption_Table.elementContainer_id.eq(id))
                .queryList();
    }
    return pageElementOptions;
}

    @Override
public void save() {
    super.save();

    //FIXME add transaction
    if(pageElementOptions !=null) {
        for(PageElementOption pao : pageElementOptions){
            pao.associateElement(this);
            pao.save();
        }
    }
}

In the save method on PageElement, the error arises when it tries to save a PageElementOption. I'm not sure how I'm supposed to fix this as I've been following the documentation. The main issue is that each time I update a Child, the parent should have the update values of its children

@ForeignKey(saveForeignKeyModel = true)
ForeignKeyContainer<MissionPage> pageContainer;

you cannot save ForeignKeyContainer as they're just glorified holders for the reference they point to. you'll have to manually save them via toModel().save()

Thank you for your response.
I'm having a problem when I try to modify a child, and then update the parent.
In this particular case, I'm setting an Option to true, after setting the option to true, I set one of the parameters of the PageElement to true, when I try saving the PageElement, I've come to realize debugging, that each child of the PageElement is saved, but with its old value.
Why is each child being saved automatically? How can I prevent this inconsistency?
Basically the code is the following in PageElement.java
    public void updateSelectedOption(int id) {
        for (PageElementOption peo : getPageElementOptions()) {
            peo.answerSelected = false;
            peo.save();
        }
        PageElementOption selectedPEO = PageElementOption.getPageElementOption(id);
        selectedPEO.answerSelected = true;
        selectedPEO.associateElement(this);
        selectedPEO.save();

        //Update that we have now answered the question
        this.isAnswered = true;
        this.update();
}

When this.update() is called, each Option is set to false again.

@OneToMany(methods = {OneToMany.Method.ALL}, variableName = "pageElementOptions")
public List<PageElementOption> getPageElementOptions() {

This line might be offending culprit. The ALL will generate a save on each item from that list of options. Potentially using stale data.

Thanks Andrew, that was indeed the problem. I changed my code a bit, now I update the object pageElementOption itself, so that there's no inconsistencies.
Thanks again

Anytime man, glad to help.
On Thu, May 5, 2016 at 2:10 PM, John Peebles<EMAIL_ADDRESS>wrote:

Thanks Andrew, that was indeed the problem. I changed my code a bit, now I
update the object pageElementOption itself, so that there's no
inconsistencies.
Thanks again
—
You are receiving this because you modified the open/close state.
Reply to this email directly or view it on GitHub
https://github.com/Raizlabs/DBFlow/issues/761#issuecomment-217229925

how to use update method remove foreign value?


Clear scratch dir upon start

Fixes #71

Codecov Report

Merging #72 into master will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master      #72   +/-   ##
=======================================
  Coverage   96.92%   96.93%
=======================================
  Files          20       20
  Lines         521      522    +1
  Branches       20       20
=======================================
+ Hits          505      506    +1
  Misses         16       16

Impacted Files
Coverage Δ

src/main/scala/AppConfig.scala
100.00% <100.00%> (ø)

src/main/scala/checker/checker.scala
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 7712b56...9cfdd81. Read the comment docs.


increase bounty payout x10

real
🆑

tweak: Bounty payouts x10

doesn't this need a cl

sure

8k for a few spears real


ci: Ubuntu updates

Fixes #1928
Test Plan

How do we know the code works?
Revert to latest, ubuntu-latest and include fixes.

Checklist

[ ] Integration tests updated

@flank-it

@pawelpasterz not sure if anything changed. But its all passing. Had issues on my fork but they dissapeared.
Maybe it needs to be merged before changes are sseen?
WDYT?

@pawelpasterz not sure if anything changed. But its all passing. Had issues on my fork but they dissapeared.
Maybe it needs to be merged before changes are sseen?
WDYT?

Yeah, changes in workflow are applied once merged to master. Let's give it a try.
One question, there are two ubuntu versions used latest and 20.04, shouldn't we use one of them?

@pawelpasterz not sure if anything changed. But its all passing. Had issues on my fork but they dissapeared.
Maybe it needs to be merged before changes are sseen?
WDYT?

Yeah, changes in workflow are applied once merged to master. Let's give it a try.
One question, there are two ubuntu versions used latest and 20.04, shouldn't we use one of them?

ubuntu-latest is ubuntu-20.04 so for now there is not difference 😄  lets stick to ubuntu-latest

@flank-it


3D Material rendering with a WorldEnvironment node fails in HTML5 export with Vulkan Mobile renderer

Godot version
4.0 beta 1 official
System information
Windows 11 (AMD Ryzen 5 3500U with Radeon Vega Mobile Gfx 2.10 GHz), Vulkan Mobile renderer
Issue description
If I make a MeshInstance3D with a StandardMaterial3D and there is a WorldEnvironment node in the scene, then it is visible when I play in the Godot Editor but does not render in an HTML5 export using the Vulkan Mobile renderer.
Steps to reproduce
To reproduce the issue from scratch do both of the following sets of steps:

Create a new project using the Vulkan Mobile renderer
Create a new 3D scene
Give the root node a child MeshInstance3D and give it a new sphere mesh
Give the root node a child Camera3D and set its Transform, Position, z to 10 (so it sees the sphere)

Set the Main scene to be that scene, and that will work as expected in both the Godot Editor and an HTML5 export but looks slightly different in the web export with the sphere appearing black instead of gray as in the Godot editor. The HTML5 export appears identical in both Chrome and Firefox.

Give the MeshInstance3D a Surface Material Override with a default StandardMaterial3D.
Give the root node a child WorldEnvironment node and give it a default New Environment.

Now the sphere is visible when the game is run from the Godot editor but is completely invisible in the HTML5 export in either Chrome or Firefox.
Minimal reproduction project
minimal.zip

HTML5 doesn't support any of the Vulkan renderers. It only supports the OpenGL renderer.
Switch to the OpenGL renderer in the project settings, so that you're actually using the same renderer on both native and HTML5.

HTML5 doesn't support any of the Vulkan renderers (there is no WebGPU renderer yet). It only supports the OpenGL renderer.

@Calinou If someone makes a new Godot 4.0 project, it defaults to Vulkan, is that correct?
If they then go to export as HTML5, Godot does not provide an option on export to use OpenGL? It will attempt to use Vulkan, appear to succeeed, but then fail to ever run?
In order to fix this, the renderer must be changed at the project-level and then exported? But if you want to also export a non-HTML5 version with Vulkan, you have to go back and change the renderer to a Vulkan?
The 3.5 version of the docs mention an option for "Export type" when exporting for HTML5 but I do not see this so I assume it is gone for 4.0?

@Nezteb You can use the per-platform override of the rendering method. In the project settings you can set renderer/rendering_method to "forward_plus" (i.e Vulkan) and set the renderer/rendering_method.web to "gl_compatibility" (i.e. Opengl3). That way you will have OpenGL3 on the web and Vulkan everywhere else. In current master these are even the default settings.